On 04/12/2012 07:09 AM, Diego Biurrun wrote:

> This fixes the warning:
> libavcodec/mpegaudiodec.c:1704:14: warning: variable ‘out_size’ set but not 
> used
> ---
>  libavcodec/mpegaudiodec.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c
> index 0ab87e1..f72e65c 100644
> --- a/libavcodec/mpegaudiodec.c
> +++ b/libavcodec/mpegaudiodec.c
> @@ -1733,6 +1733,10 @@ static int decode_frame_adu(AVCodecContext *avctx, 
> void *data,
>      s->frame_size = len;
>  
>      out_size = mp_decode_frame(s, NULL, buf, buf_size);
> +    if (out_size < 0) {
> +        av_log(avctx, AV_LOG_ERROR, "Error while decoding MPEG audio 
> frame.\n");
> +        return AVERROR_INVALIDDATA;
> +    }
>  
>      *got_frame_ptr   = 1;
>      *(AVFrame *)data = s->frame;


looks ok.

-Justin

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

Reply via email to