[MPlayer-dev-eng] [patch] backports/fixes from uoti demux_mkv.c for comp_algo==3
Aurelien Jacobs
aurel at gnuage.org
Sat Aug 14 22:39:38 CEST 2010
On Sat, Aug 14, 2010 at 08:23:42AM +0200, Reimar Döffinger wrote:
> On Fri, Aug 13, 2010 at 12:27:13PM +0200, Aurelien Jacobs wrote:
> > demux_mkv.c is full of potential mem leak, integer overflow, null
> > pointer deference and so on.
> > Fixing them all would be quite some work (IMHO not worth at all).
> > Fixing just one is not much helpfull...
>
> I think he fixed quite a few issues, IMO it's a good idea to do,
> also because unfortunately some distributions still ship with
> old code where
> 1) demux_mkv is the default
> 2) libavformat is too old for -demuxer lavf to be usable.
I am not opposed to the patch being applied. I just won't waste my time
reviewing it.
Aurel
More information about the MPlayer-dev-eng
mailing list