[FFmpeg-devel] [PATCH] wvenc: multichannel muxing

[Paul B Mahol]

Signed-off-by: Paul B Mahol <onemda at gmail.com>
---
 libavformat/wvenc.c | 54 +++++++++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 48 insertions(+), 6 deletions(-)

diff --git a/libavformat/wvenc.c b/libavformat/wvenc.c
index d225ed4..2d08a01 100644
--- a/libavformat/wvenc.c
+++ b/libavformat/wvenc.c
@@ -19,19 +19,21 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
+#include "libavutil/intreadwrite.h"
 #include "avformat.h"
 #include "internal.h"
 #include "avio_internal.h"
 #include "apetag.h"
 
+#define WV_EXTRA_SIZE 12
+#define WV_END_BLOCK  0x1000
+
 typedef struct{
     uint32_t duration;
-    int off;
 } WVMuxContext;
 
 static int write_header(AVFormatContext *s)
 {
-    WVMuxContext *wc = s->priv_data;
     AVCodecContext *codec = s->streams[0]->codec;
 
     if (s->nb_streams > 1) {
@@ -46,7 +48,6 @@ static int write_header(AVFormatContext *s)
         av_log_missing_feature(s, "remuxing from matroska container", 0);
         return AVERROR_PATCHWELCOME;
     }
-    wc->off = codec->channels > 2 ? 4 : 0;
     avpriv_set_pts_info(s->streams[0], 64, 1, codec->sample_rate);
 
     return 0;
@@ -55,18 +56,59 @@ static int write_header(AVFormatContext *s)
 static int write_packet(AVFormatContext *s, AVPacket *pkt)
 {
     WVMuxContext *wc = s->priv_data;
+    AVCodecContext *codec = s->streams[0]->codec;
     AVIOContext *pb = s->pb;
+    uint64_t size;
+    uint32_t flags;
+    uint32_t left = pkt->size;
+    uint8_t *ptr = pkt->data;
+    int off = codec->channels > 2 ? 4 : 0;
 
     wc->duration += pkt->duration;
     ffio_wfourcc(pb, "wvpk");
-    avio_wl32(pb, pkt->size + 12 + wc->off);
+    if (off) {
+        size = AV_RL32(pkt->data);
+        if (size <= 12)
+            return AVERROR_INVALIDDATA;
+        size -= 12;
+    } else {
+        size = pkt->size;
+    }
+
+    if (size + off > left)
+        return AVERROR_INVALIDDATA;
+
+    avio_wl32(pb, size + 12);
     avio_wl16(pb, 0x410);
     avio_w8(pb, 0);
     avio_w8(pb, 0);
     avio_wl32(pb, -1);
     avio_wl32(pb, pkt->pts);
-    avio_write(s->pb, pkt->data, pkt->size);
-    avio_flush(s->pb);
+    ptr += off; left -= off;
+    flags = AV_RL32(ptr + 4);
+    avio_write(pb, ptr, size);
+    ptr += size; left -= size;
+
+    while (!(flags & WV_END_BLOCK) &&
+            (left >= 4 + WV_EXTRA_SIZE)) {
+        ffio_wfourcc(pb, "wvpk");
+        size = AV_RL32(ptr);
+        ptr += 4; left -= 4;
+        if (size < 24 || size - 24 > left)
+            return AVERROR_INVALIDDATA;
+        avio_wl32(pb, size);
+        avio_wl16(pb, 0x410);
+        avio_w8(pb, 0);
+        avio_w8(pb, 0);
+        avio_wl32(pb, -1);
+        avio_wl32(pb, pkt->pts);
+        flags = AV_RL32(ptr + 4);
+        avio_write(pb, ptr, WV_EXTRA_SIZE);
+        ptr += WV_EXTRA_SIZE; left -= WV_EXTRA_SIZE;
+        avio_write(pb, ptr, size - 24);
+        ptr += size - 24; left -= size - 24;
+    }
+    avio_flush(pb);
 
     return 0;
 }
-- 
1.7.11.2