mpegaudiodec: Do not discard mp_decode_frame() return value.
authorDiego Biurrun <diego@biurrun.de>
Thu, 5 Apr 2012 14:48:33 +0000 (16:48 +0200)
committerDiego Biurrun <diego@biurrun.de>
Wed, 18 Apr 2012 16:28:24 +0000 (18:28 +0200)
This fixes the warning:
libavcodec/mpegaudiodec.c:1704:14: warning: variable ‘out_size’ set but not used

libavcodec/mpegaudiodec.c

index 0ab87e1..f72e65c 100644 (file)
@@ -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;