[FFmpeg-cvslog] lavf/mp3enc: don't abort if audio packets can't be buffered

James Almer git at videolan.org
Wed Aug 27 19:01:24 CEST 2014


ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Tue Aug 26 21:31:14 2014 -0300| [d3e9bee6f7e527dcb15ca87a426a09ad9f5a8a60] | committer: James Almer

lavf/mp3enc: don't abort if audio packets can't be buffered

Instead skip the picture streams and flush the queued audio packets.
Should address ticket #3851.

Signed-off-by: James Almer <jamrial at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d3e9bee6f7e527dcb15ca87a426a09ad9f5a8a60
---

 libavformat/mp3enc.c |    8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/libavformat/mp3enc.c b/libavformat/mp3enc.c
index c0270c2..7e6b2e1 100644
--- a/libavformat/mp3enc.c
+++ b/libavformat/mp3enc.c
@@ -421,8 +421,12 @@ static int mp3_write_packet(AVFormatContext *s, AVPacket *pkt)
             /* buffer audio packets until we get all the pictures */
             AVPacketList *pktl = av_mallocz(sizeof(*pktl));
             int ret;
-            if (!pktl)
-                return AVERROR(ENOMEM);
+            if (!pktl) {
+                av_log(s, AV_LOG_WARNING, "Not enough memory to buffer audio. Skipping picture streams\n");
+                mp3->pics_to_write = 0;
+                mp3_queue_flush(s);
+                return mp3_write_audio_packet(s, pkt);
+            }
 
             ret = av_copy_packet(&pktl->pkt, pkt);
             if (ret < 0) {



More information about the ffmpeg-cvslog mailing list