On Tue, May 12, 2015 at 05:55:38PM +0200, Luca Barbato wrote:
> --- a/libavcodec/ratecontrol.c
> +++ b/libavcodec/ratecontrol.c
> @@ -932,6 +934,11 @@ static int init_pass2(MpegEncContext *s)
>
> qscale = av_malloc(sizeof(double) * rcc->num_entries);
> blurred_qscale = av_malloc(sizeof(double) * rcc->num_entries);
> + if (!qscale || !blurred_qscale) {
> + av_freep(&qscale);
> + av_freep(&blurred_qscale);
> + return AVERROR(ENOMEM);
> + }
av_freep is unnecessary here, these are function-local variables. You
had plain av_free() here in v5 at least.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
