mp3enc: fix a triggerable assert
authorAnton Khirnov <anton@khirnov.net>
Fri, 14 Nov 2014 19:20:50 +0000 (20:20 +0100)
committerAnton Khirnov <anton@khirnov.net>
Sat, 15 Nov 2014 07:51:50 +0000 (08:51 +0100)
We have to check against the number of bytes actually needed, not the
theoretical maximum size.

(cherry picked from commit 12700b0219521a5f20c8ba47b3ad7857ea9e0554)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 871d99ef77336069e5a8ece947c8160d9bc4d5ea)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
libavformat/mp3enc.c

index 631705c..0596d60 100644 (file)
@@ -195,7 +195,7 @@ static void mp3_write_xing(AVFormatContext *s)
 
     avpriv_mpegaudio_decode_header(&mpah, header);
 
-    av_assert0(mpah.frame_size >= XING_MAX_SIZE);
+    av_assert0(mpah.frame_size >= bytes_needed);
 
     ffio_fill(s->pb, 0, xing_offset);
     mp3->xing_offset = avio_tell(s->pb);