On Wed, Oct 29, 2014 at 8:51 PM, Vittorio Giovara
<[email protected]> wrote:
> CC: [email protected]
> Bug-Id: CID 29254
> ---
>  libavcodec/ratecontrol.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/ratecontrol.c b/libavcodec/ratecontrol.c
> index e18878d..08f5ccc 100644
> --- a/libavcodec/ratecontrol.c
> +++ b/libavcodec/ratecontrol.c
> @@ -176,7 +176,7 @@ av_cold int ff_rate_control_init(MpegEncContext *s)
>
>          /* read stats */
>          p = s->avctx->stats_in;
> -        for (i = 0; i < rcc->num_entries - s->max_b_frames; i++) {
> +        for (i = 0; i < rcc->num_entries - s->max_b_frames && p; i++) {
>              RateControlEntry *rce;
>              int picture_number;
>              int e;

This can't actually happen since input is already validated in the "/*
find number of pics */" block.
Marked as false positive.
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to