[MPlayer-dev-eng] [BUG][POSSIBLE FIX]Sig11 in libmpdemux/demuxer.c for files with wrong extension

[Arpi]

Hi,

> Hi, I have uploaded a file (demuxer-detection-segfault.m2v.bz2) which
> causes mplayer (current CVS  version) to crash while trying to detect the
> demuxer.
> I attach a gdb log and the commands used to generate it.
> ~/mldonkey-distrib-2.02-0/temp/mplayer-crash.m2v is the same file as the

i could reproduce the bug (i'll fix it), but this file doens't look like
a valid mpeg2 video stream. it does't look like mpeg stream at all.

> bunzip2 of the file I has uploaded. A simple fix, which will cause huge
> memory leaks is to check just before the line of the crash that
> d_video->sh is not NULL, and if it is, return NULL as the demuxer, but
> this is very ugly hack, that will hide te problem instead of solving it.

yes, it's not the right way.


A'rpi / Astral & ESP-team

--
Developer of MPlayer G2, the Movie Framework for all - http://www.MPlayerHQ.hu