[FFmpeg-devel] [PATCH] snow_dwt: Don't try and free members of non-existent arrays

Michael Niedermayer michaelni at gmx.at
Tue Dec 16 03:50:27 CET 2014


On Mon, Dec 15, 2014 at 10:57:28PM +0000, Derek Buitenhuis wrote:
> If allocation fails earlier on, and thd next frame is processed,
> the slice buffer could be left in a state where line and data_stack
> have already been freed, or are otherwise null pointers.
> 
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> ---
>  libavcodec/snow_dwt.c | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)

LGTM

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141216/6e77c671/attachment.asc>


More information about the ffmpeg-devel mailing list