[MPlayer-dev-eng] [PATCH] fix bug in free_demuxer

Meihui Fan mhfan at hhcn.com
Fri Aug 19 03:18:18 CEST 2005



Rich Felker wrote:
> On Thu, Aug 18, 2005 at 10:27:30PM +0800, Meihui Fan wrote:
> 
>>Let's make all demuxer freed in free_demuxer,
>>so we don't have to access the member(demux->desc->type) of the demuxer
>>which already freed in libmpdemux/demux_demuxer.c/demux_close_demuxers.
>>
>>This bug may cause the MPlayer crashed when playing separated video and 
>>audio file.
> 
> 
> Your patch is what causes MPlayer to crash, by accessing already-freed
> memory. The way it is now is correct as far as I can tell.
Just try this:
$ mplayer foo.avi -audiofile bar.mp3
then press `q'.
> 
> Rich
> 
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
> 

-- 
Best Regards,
范美辉 (Meihui Fan)
Software Engineering Department
Tel: +86-551-5333155/156/157,5325173
FAX: +86-551-5325323
Products Info. : http://www.hhcn.com
Tech. Support  : http://bbs.hhcn.com
HHTech : An Embedded Linux Tech. Provider in Mainland China




More information about the MPlayer-dev-eng mailing list