[FFmpeg-devel] [PATCH] avformat/mp3enc: Assert that the header we assembled is valid

Michael Niedermayer michael at niedermayer.cc
Thu Feb 4 00:20:51 CET 2016


Fixes: CID1351343

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
 libavformat/mp3enc.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavformat/mp3enc.c b/libavformat/mp3enc.c
index 7296234..71f5178 100644
--- a/libavformat/mp3enc.c
+++ b/libavformat/mp3enc.c
@@ -192,7 +192,8 @@ static int mp3_write_xing(AVFormatContext *s)
             return -1;
         header |= mask;
 
-        avpriv_mpegaudio_decode_header(&mpah, header);
+        ret = avpriv_mpegaudio_decode_header(&mpah, header);
+        av_assert0(ret >= 0);
         mp3->xing_offset = xing_offtbl[mpah.lsf == 1][mpah.nb_channels == 1] + 4;
         bytes_needed     = mp3->xing_offset + XING_SIZE;
 
-- 
1.7.9.5



More information about the ffmpeg-devel mailing list