On 06/08/14 12:57, Anton Khirnov wrote:
> 0 means no data consumed, so it can trigger an infinite loop in the
> caller.
> 
> CC:[email protected]
> ---
>  libavcodec/cdgraphics.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavcodec/cdgraphics.c b/libavcodec/cdgraphics.c
> index 752003f..c3e42e7 100644
> --- a/libavcodec/cdgraphics.c
> +++ b/libavcodec/cdgraphics.c
> @@ -349,10 +349,9 @@ static int cdg_decode_frame(AVCodecContext *avctx,
>          *got_frame = 1;
>      } else {
>          *got_frame = 0;
> -        buf_size   = 0;
>      }
>  
> -    return buf_size;
> +    return avpkt->size;
>  }
>  
>  static av_cold int cdg_decode_end(AVCodecContext *avctx)
> 

Looks fine to me.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to