tag: id3v2: fix parsing of unsynced frames with data length indicator
authorTim-Philipp Müller <tim.muller@collabora.co.uk>
Tue, 30 Mar 2010 00:50:32 +0000 (01:50 +0100)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Sun, 14 Aug 2011 23:10:34 +0000 (00:10 +0100)
Fixes bug #614158.

gst-libs/gst/tag/id3v2frames.c

index 79fdf61..021a3d1 100644 (file)
@@ -116,9 +116,11 @@ id3demux_id3v2_parse_frame (ID3TagsWorking * work)
     }
     frame_data += 4;
     frame_data_size -= 4;
-    if (work->parse_size < frame_data_size) {
-      GST_WARNING ("ID3v2 frame %s has invalid size %d.", tag_name,
-          frame_data_size);
+    GST_LOG ("Un-unsynced data size %d (of %d)", work->parse_size,
+        frame_data_size);
+    if (work->parse_size > frame_data_size) {
+      GST_WARNING ("ID3v2 frame %s data has invalid size %d (>%d)",
+          work->frame_id, work->parse_size, frame_data_size);
       return FALSE;
     }
   }