Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[security]memory leak in MP4Box infe_box_read #1786

Closed
5n1p3r0010 opened this issue May 8, 2021 · 0 comments
Closed

[security]memory leak in MP4Box infe_box_read #1786

5n1p3r0010 opened this issue May 8, 2021 · 0 comments

Comments

@5n1p3r0010
Copy link

Hi,

There is a memory leak issue in gpac MP4Box infe_box_read,this can reproduce on the lattest commit.

Steps To Reproduce

build:

CC=gcc CXX=g++ CFLAGS="-fsanitize=address" CXXFLAGS="-fsanitize=address" LDFLAGS="-fsanitize=address" ./configure --enable-debug
make

run as:

MP4Box -info <poc>

shows the following log:

=================================================================
==3638776==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 33 byte(s) in 4 object(s) allocated from:
    #0 0x7faaab471bc8 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.5+0x10dbc8)
    #1 0x7faaaa9aedcb in gf_malloc utils/alloc.c:150
    #2 0x7faaaab9fa3e in infe_box_read isomedia/box_code_meta.c:570
    #3 0x7faaaabb9748 in gf_isom_box_read isomedia/box_funcs.c:1796
    #4 0x7faaaabb804b in gf_isom_box_parse_ex isomedia/box_funcs.c:261
    #5 0x7faaaabb74ec in gf_isom_parse_root_box isomedia/box_funcs.c:38
    #6 0x7faaaabc1e04 in gf_isom_parse_movie_boxes_internal isomedia/isom_intern.c:318
    #7 0x7faaaabc3708 in gf_isom_parse_movie_boxes isomedia/isom_intern.c:777
    #8 0x7faaaabc6922 in gf_isom_open_progressive_ex isomedia/isom_read.c:467
    #9 0x7faaaabc69d2 in gf_isom_open_progressive isomedia/isom_read.c:493
    #10 0x5613aba3bee1 in mp4boxMain /home/r00t/fuzz/target/tmp/gpac/applications/mp4box/main.c:5724
    #11 0x5613aba3e653 in main /home/r00t/fuzz/target/tmp/gpac/applications/mp4box/main.c:6333
    #12 0x7faaaa7280b2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x270b2)

Direct leak of 33 byte(s) in 4 object(s) allocated from:
    #0 0x7faaab471bc8 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.5+0x10dbc8)
    #1 0x7faaaa9aedcb in gf_malloc utils/alloc.c:150
    #2 0x7faaaab9fa3e in infe_box_read isomedia/box_code_meta.c:570
    #3 0x7faaaabb9748 in gf_isom_box_read isomedia/box_funcs.c:1796
    #4 0x7faaaabb804b in gf_isom_box_parse_ex isomedia/box_funcs.c:261
    #5 0x7faaaabb74ec in gf_isom_parse_root_box isomedia/box_funcs.c:38
    #6 0x7faaaabc1e04 in gf_isom_parse_movie_boxes_internal isomedia/isom_intern.c:318
    #7 0x7faaaabc3708 in gf_isom_parse_movie_boxes isomedia/isom_intern.c:777
    #8 0x7faaaabc3a9b in gf_isom_open_file isomedia/isom_intern.c:897
    #9 0x7faaaabc6a41 in gf_isom_open isomedia/isom_read.c:509
    #10 0x5613aba3be7e in mp4boxMain /home/r00t/fuzz/target/tmp/gpac/applications/mp4box/main.c:5720
    #11 0x5613aba3e653 in main /home/r00t/fuzz/target/tmp/gpac/applications/mp4box/main.c:6333
    #12 0x7faaaa7280b2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x270b2)

SUMMARY: AddressSanitizer: 66 byte(s) leaked in 8 allocation(s).

Reporter:

5n1p3r0010 from Topsec Alpha Lab
infe_box_read.zip

@jeanlf jeanlf closed this as completed in ec64c7b May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant