mp3enc: fix playback with mp3 encoded files on some devices.
authorClément Bœsch <clement.boesch@smartjog.com>
Mon, 2 Jan 2012 08:39:11 +0000 (09:39 +0100)
committerClément Bœsch <ubitux@gmail.com>
Mon, 2 Jan 2012 14:34:03 +0000 (15:34 +0100)
This partially reverts 1fea432b; header variable needs to keep its value
in case it triggers "if (needed <= c.frame_size)".

libavformat/mp3enc.c

index 42fc39d..8d82452 100644 (file)
@@ -160,7 +160,7 @@ static int mp3_write_xing(AVFormatContext *s)
     int              best_bitrate_idx;
     int              best_bitrate_error= INT_MAX;
     int64_t          xing_offset;
-    int32_t          header;
+    int32_t          header, mask;
     MPADecodeHeader  c;
     int              srate_idx, i, channels;
     int              needed;
@@ -200,8 +200,9 @@ static int mp3_write_xing(AVFormatContext *s)
     for (bitrate_idx= best_bitrate_idx;; bitrate_idx++) {
         if (15 == bitrate_idx)
             return -1;
-
-        avpriv_mpegaudio_decode_header(&c, header | (bitrate_idx << (4+8)));
+        mask = bitrate_idx << (4+8);
+        header |= mask;
+        avpriv_mpegaudio_decode_header(&c, header);
         xing_offset=xing_offtbl[c.lsf == 1][c.nb_channels == 1];
         needed = 4              // header
                + xing_offset
@@ -213,6 +214,7 @@ static int mp3_write_xing(AVFormatContext *s)
 
         if (needed <= c.frame_size)
             break;
+        header &= ~mask;
     }
 
     avio_wb32(s->pb, header);