changeset 2668:8d400e382d6e libavformat

Prevent mov muxer from writing empty udta atoms.
author benoit
date Wed, 24 Oct 2007 14:20:15 +0000
parents e896ac505ec6
children 09eb62fa2a0e
files movenc.c
diffstat 1 files changed, 20 insertions(+), 8 deletions(-) [+]
line wrap: on
line diff
--- a/movenc.c	Wed Oct 24 04:56:22 2007 +0000
+++ b/movenc.c	Wed Oct 24 14:20:15 2007 +0000
@@ -1220,8 +1220,23 @@
 static int mov_write_udta_tag(ByteIOContext *pb, MOVContext* mov,
                               AVFormatContext *s)
 {
+    int i, req = 0;
+
+    /* Requirements */
+    for (i=0; i<mov->nb_streams; i++) {
+        if(mov->tracks[i].entry <= 0) continue;
+        if (mov->tracks[i].enc->codec_id == CODEC_ID_AAC ||
+            mov->tracks[i].enc->codec_id == CODEC_ID_MPEG4) {
+            req = 1;
+            break;
+        }
+    }
+
+    if (s->title[0]   || s->author[0] || s->album[0] || s->year ||
+        s->comment[0] || s->genre[0]  || s->track ||
+       (mov->mode == MODE_MOV &&
+      ((mov->tracks[0].enc && !mov->tracks[0].enc->flags & CODEC_FLAG_BITEXACT) || req))) {
     offset_t pos = url_ftell(pb);
-    int i;
 
     put_be32(pb, 0); /* size */
     put_tag(pb, "udta");
@@ -1231,14 +1246,8 @@
 
     if(mov->mode == MODE_MOV){ // the title field breaks gtkpod with mp4 and my suspicion is that stuff is not valid in mp4
         /* Requirements */
-        for (i=0; i<mov->nb_streams; i++) {
-            if(mov->tracks[i].entry <= 0) continue;
-            if (mov->tracks[i].enc->codec_id == CODEC_ID_AAC ||
-                mov->tracks[i].enc->codec_id == CODEC_ID_MPEG4) {
+        if (req)
                 mov_write_string_tag(pb, "\251req", "QuickTime 6.0 or greater", 0);
-                break;
-            }
-        }
 
         mov_write_string_tag(pb, "\251nam", s->title         , 0);
         mov_write_string_tag(pb, "\251aut", s->author        , 0);
@@ -1251,6 +1260,9 @@
     }
 
     return updateSize(pb, pos);
+    }
+
+    return 0;
 }
 
 static int utf8len(uint8_t *b){