[FFmpeg-devel] [PATCH] MP3: ID3V2 corrupted => bad offset
Yvan Labadie
ylabadie
Fri Dec 5 16:08:11 CET 2008
M?ns Rullg?rd a ?crit :
> Michael Niedermayer wrote:
>
>> On Fri, Dec 05, 2008 at 02:18:03PM -0000, M?ns Rullg?rd wrote:
>>
>>> [...]
>>> ID3v2.3 and earlier used a plain integer. The "sync safe" encoding was
>>> introduced in v2.4.
>>>
>> that explains it, assuming all the failing files are < 2.4
>> a proper solution would then be to fix <2.4 support
>>
Ooh! that kind of explains everything... :D
So I join a patch that uses the conversion corresponding to the version
with a simple switch/case...
> That said, I wouldn't be very surprised if there are files out there
> with any kind of errors. We're talking about mp3 files afters all...
>
Sure, so I should still do a check of mp3_parse_vbr_tags() and correct
the offset if it fails... like we discuss in previous mail from Michael.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch_mp3.diff
Type: text/x-patch
Size: 928 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081205/0d9abc23/attachment.bin>
More information about the ffmpeg-devel
mailing list