search

justdan96 / tsMuxer

tsMuxer is a transport stream muxer for remuxing/muxing elementary streams, EVO/VOB/MPG, MKV/MKA, MP4/MOV, TS, M2TS to TS to M2TS. Supported video codecs H.264/AVC, H.265/HEVC, VC-1, MPEG2. Supported audio codecs AAC, AC3 / E-AC3(DD+), DTS/ DTS-HD.
Apache License 2.0
830 stars 140 forks source link

SEGV in ./tsMuxer/tsMuxer/vvc.cpp:283:32 in VvcVpsUnit::deserialize() #783

Closed gandalf4a closed 9 months ago

gandalf4a commented 9 months ago

version

$ git show
commit 2539d074cd4da0547b97aedd8bc12252b973907c (HEAD -> master, tag: nightly-2023-10-05-01-55-56, origin/master, origin/HEAD)
Author: jcdr428 <jessiedeer@hotmail.com>
Date:   Wed Oct 4 10:17:02 2023 +0100

system

$ uname -a
Linux user-GE40-2PC-Dragon-Eyes 6.2.0-33-generic #33~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Thu Sep  7 10:33:52 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

asan

tsMuxeR version git-2539d07. github.com/justdan96/tsMuxer
AddressSanitizer:DEADLYSIGNAL
=================================================================
==542439==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x55c0fbd17252 bp 0x7fffd315d830 sp 0x7fffd315d300 T0)
==542439==The signal is caused by a WRITE memory access.
==542439==Hint: address points to the zero page.
    #0 0x55c0fbd17252 in VvcVpsUnit::deserialize() /home/user/fuzzing_tsMuxer/tsMuxer/tsMuxer/vvc.cpp:283:32
    #1 0x55c0fbd3b813 in VVCStreamReader::checkStream(unsigned char*, int) /home/user/fuzzing_tsMuxer/tsMuxer/tsMuxer/vvcStreamReader.cpp:60:24
    #2 0x55c0fba1a7bf in METADemuxer::detectTrackReader(unsigned char*, int, AbstractStreamReader::ContainerType, int, int) /home/user/fuzzing_tsMuxer/tsMuxer/tsMuxer/metaDemuxer.cpp:791:21
    #3 0x55c0fba0d766 in METADemuxer::DetectStreamReader(BufferedReaderManager const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, bool) /home/user/fuzzing_tsMuxer/tsMuxer/tsMuxer/metaDemuxer.cpp:685:35
    #4 0x55c0fb932a7a in detectStreamReader(char const*, MPLSParser*, bool) /home/user/fuzzing_tsMuxer/tsMuxer/tsMuxer/main.cpp:114:34
    #5 0x55c0fb946efb in main /home/user/fuzzing_tsMuxer/tsMuxer/tsMuxer/main.cpp:689:17
    #6 0x7f92fac29d8f in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #7 0x7f92fac29e3f in __libc_start_main csu/../csu/libc-start.c:392:3
    #8 0x55c0fb6cdd24 in _start (/home/user/fuzzing_tsMuxer/tsMuxer/build/tsMuxer/tsmuxer+0x249d24) (BuildId: 93aa533ae68cbad6d874b6199ee386d19d3a575e)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /home/user/fuzzing_tsMuxer/tsMuxer/tsMuxer/vvc.cpp:283:32 in VvcVpsUnit::deserialize()
==542439==ABORTING

reproduce

./tsmuxer poc

POC File

https://github.com/gandalf4a/crash_report/blob/main/tsMuxer/poc_segv_vvc

Credit

Gandalf4a
jcdr428 commented 9 months ago

Thanks @gandalf4a