[FFmpeg-devel] [PATCH] check for null pointer in nuv decoder (issue 2524)

Daniel Kang daniel.d.kang
Tue Jan 11 01:15:07 CET 2011


On Mon, Jan 10, 2011 at 2:30 AM, Reimar D?ffinger
<Reimar.Doeffinger at gmx.de>wrote:

>  On Sun, Jan 09, 2011 at 10:32:02PM -0500, Daniel Kang wrote:
> > In decode_init in the nuv decoder, c->decomp_buf is set to NULL. In most
> > cases, this is okay, as the buffer is reallocated. However, for invalid
> > nuv files, ffmpeg does not reallocate the buffer. In this case, ffmpeg
> > will crash when trying to access the null pointer. The patch attached
> > adds a check for this.
>
> There are other issues, but please try just setting decomp_size to 0
> when allocation fails, that should be enough.


I have tried only setting decomp_size (and outlen) to 0, but ffmpeg
still crashes. av_lzo1x_decode still attempts to write to out even with
an invalid buffer.



More information about the ffmpeg-devel mailing list