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.
This commit is contained in:
parent
891ed455e7
commit
09b26dbf5c
@ -108,7 +108,11 @@ id3demux_id3v2_parse_frame (ID3TagsWorking * work)
|
|||||||
ID3V2_FRAME_FORMAT_DATA_LENGTH_INDICATOR)) {
|
ID3V2_FRAME_FORMAT_DATA_LENGTH_INDICATOR)) {
|
||||||
if (work->hdr.frame_data_size <= 4)
|
if (work->hdr.frame_data_size <= 4)
|
||||||
return FALSE;
|
return FALSE;
|
||||||
|
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);
|
work->parse_size = read_synch_uint (frame_data, 4);
|
||||||
|
}
|
||||||
frame_data += 4;
|
frame_data += 4;
|
||||||
frame_data_size -= 4;
|
frame_data_size -= 4;
|
||||||
if (work->parse_size < frame_data_size) {
|
if (work->parse_size < frame_data_size) {
|
||||||
|
Loading…
x
Reference in New Issue
Block a user