On Wed, Dec 10, 2014 at 12:43 AM, Luca Barbato <[email protected]> wrote:
> Prevent an uninitialized data access.
>
> CC: [email protected]
> Bug-Id: CID 703824 / CID 703825
> Signed-off-by: Vittorio Giovara <[email protected]>
> Signed-off-by: Luca Barbato <[email protected]>
> ---
>
> W/out the pointless array initialization.
>
>  libavcodec/proresenc.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/proresenc.c b/libavcodec/proresenc.c
> index 3a82c2c..f61aa60 100644
> --- a/libavcodec/proresenc.c
> +++ b/libavcodec/proresenc.c
> @@ -820,10 +820,9 @@ static int find_slice_quant(AVCodecContext *avctx, const 
> AVFrame *pic,
>          if (ctx->alpha_bits)
>              bits += estimate_alpha_plane(ctx, &error, src, linesize[3],
>                                           mbs_per_slice, q, td->blocks[3]);
> -        if (bits > 65000 * 8) {
> +        if (bits > 65000 * 8)
>              error = SCORE_LIMIT;
> -            break;
> -        }
> +
>          slice_bits[q]  = bits;
>          slice_score[q] = error;
>      }
> --
> 2.1.0

lgtm

-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to