I don't know if this is necessary or relevant, but since you said snow encoding failed, and valgrind reported that the tag was uninitialised, why not clear the whole of it.

Mats

--
Mats Peterson
http://matsp888.no-ip.org/~mats/
>From a1d51457e12a488b7ad993a61579cb0ba3bdd9f9 Mon Sep 17 00:00:00 2001
From: Mats Peterson <matsp...@yahoo.com>
Date: Sun, 13 Mar 2016 16:09:45 +0100
Subject: [PATCH] lavf/avienc: Clear whole tag in avi_add_ientry()

---
 libavformat/avienc.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/avienc.c b/libavformat/avienc.c
index dceb2ef..7a7abb8 100644
--- a/libavformat/avienc.c
+++ b/libavformat/avienc.c
@@ -124,7 +124,7 @@ static int avi_add_ientry(AVFormatContext *s, int stream_index, char *tag,
     if (tag)
         memcpy(idx->cluster[cl][id].tag, tag, 4);
     else
-        *(idx->cluster[cl][id].tag) = '\0';
+        memset(idx->cluster[cl][id].tag, 0, 4);
     idx->cluster[cl][id].flags = flags;
     idx->cluster[cl][id].pos   = avio_tell(pb) - avi->movi_list;
     idx->cluster[cl][id].len   = size;
-- 
1.7.10.4

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to