diff options
author | Jan Schmidt <thaytan@mad.scientist.com> | 2005-12-28 18:55:32 +0000 |
---|---|---|
committer | Jan Schmidt <thaytan@mad.scientist.com> | 2005-12-28 18:55:32 +0000 |
commit | 041a4a0d89ceac910acf0de86d4e68fa2762e3b2 (patch) | |
tree | 654468c1844ea83d3cdc7f96a8e2451de201a932 /gst/id3demux/id3tags.c | |
parent | 0c2414650ac588b947dd9761394fdd28b65c3be2 (diff) |
gst/id3demux/id3tags.c: If a broken tag has 0 bytes payload, at least still skip the 10 byte header
Original commit message from CVS:
* gst/id3demux/id3tags.c: (id3demux_read_id3v2_tag):
If a broken tag has 0 bytes payload, at least still skip
the 10 byte header
Diffstat (limited to 'gst/id3demux/id3tags.c')
-rw-r--r-- | gst/id3demux/id3tags.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/gst/id3demux/id3tags.c b/gst/id3demux/id3tags.c index 1cc8c14e..e25b4291 100644 --- a/gst/id3demux/id3tags.c +++ b/gst/id3demux/id3tags.c @@ -143,9 +143,12 @@ id3demux_read_id3v2_tag (GstBuffer * buffer, guint * id3v2_size, /* Read the size from the header */ read_size = read_synch_uint (data + 6, 4); if (read_size == 0) { + /* Tag has no frames attached. Ignore it, but skip the header */ + if (id3v2_size) + *id3v2_size = ID3V2_HDR_SIZE; return ID3TAGS_BROKEN_TAG; } - read_size += 10; + read_size += ID3V2_HDR_SIZE; /* Expand the read size to include a footer if there is one */ if (flags & ID3V2_HDR_FLAG_FOOTER) { |