On 9/16/2017 9:42 PM, Michael Niedermayer wrote:
> Fixes: memleak
> Fixes: 3203/clusterfuzz-testcase-minimized-4514553595428864
> 
> Found-by: continuous fuzzing process 
> https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
> ---
>  libavcodec/pngdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
> index dce8faf168..0d6612ccca 100644
> --- a/libavcodec/pngdec.c
> +++ b/libavcodec/pngdec.c
> @@ -1414,7 +1414,7 @@ static int decode_frame_png(AVCodecContext *avctx,
>      }
>  
>      if ((ret = av_frame_ref(data, s->picture.f)) < 0)
> -        return ret;
> +        goto the_end;
>  
>      *got_frame = 1;
>  

LGTM.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to