[FFmpeg-devel] [PATCH 06/19] movtextdec: fix return value for too small packets.

Philip Langdale philipl at overt.org
Mon Jul 30 05:18:57 CEST 2012


On Sun, 29 Jul 2012 15:57:24 +0200
Nicolas George <nicolas.george at normalesup.org> wrote:

> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  libavcodec/movtextdec.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/movtextdec.c b/libavcodec/movtextdec.c
> index 43ebb78..b34523d 100644
> --- a/libavcodec/movtextdec.c
> +++ b/libavcodec/movtextdec.c
> @@ -66,7 +66,7 @@ static int mov_text_decode_frame(AVCodecContext
> *avctx, const char *end;
>  
>      if (!ptr || avpkt->size <= 2)
> -        return 0;
> +        return avpkt->size ? AVERROR_INVALIDDATA : 0;
>  
>      /*
>       * The first two bytes of the packet are the length of the text
> string

Looks good to me.

--phil


More information about the ffmpeg-devel mailing list