From 09b26dbf5ca4a14e5cc53529546d71c1509f3c94 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim-Philipp=20M=C3=BCller?= Date: Fri, 7 Aug 2009 16:42:39 +0100 Subject: [PATCH] tag: id3v2: sizes in ID3 v2.3 are unlikely to be sync-safe integers In ID3 v2.3 compressed frames will have a 4-byte data length indicator after the frame header to indicate the size of the decompressed data. This integer is unlikely to be a sync-safe integer for v2.3 tags, only in v2.4 it's sync-safe. --- gst-libs/gst/tag/id3v2frames.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/gst-libs/gst/tag/id3v2frames.c b/gst-libs/gst/tag/id3v2frames.c index a0331ef385..80a123a27a 100644 --- a/gst-libs/gst/tag/id3v2frames.c +++ b/gst-libs/gst/tag/id3v2frames.c @@ -108,7 +108,11 @@ id3demux_id3v2_parse_frame (ID3TagsWorking * work) ID3V2_FRAME_FORMAT_DATA_LENGTH_INDICATOR)) { if (work->hdr.frame_data_size <= 4) return FALSE; - work->parse_size = read_synch_uint (frame_data, 4); + if (ID3V2_VER_MAJOR (work->hdr.version) == 3) { + work->parse_size = GST_READ_UINT32_BE (frame_data); + } else { + work->parse_size = read_synch_uint (frame_data, 4); + } frame_data += 4; frame_data_size -= 4; if (work->parse_size < frame_data_size) {