Bug #3742 for MP3-Info: Incompatiblity between APIC & TXXX/WXXX frames

Sun Sep 07 08:54:12 2003 Guest - Ticket created

Subject:

Incompatiblity between APIC & TXXX/WXXX frames

Hi, I've been experiencing for a while with MP3::Info, and this morning I found out that when an APIC frame is included in a file, the TXXX and WXXX frames are no longer reported by get_mp3tags(). Get rid of the APIC frame, and they're back... Attached is a sample MP3 file with an APIC frame (I use ID3-TagIt to edit the ID3 v2.4 tag). I'm using Perl 5.8.0, build 806 from ActiveState, under Win2000/SP3. Thanks, Pyt.

Download test APIC.mp3
audio/mpeg 297.2k

Message body not shown because it is not plain text.

Mon Sep 08 06:09:50 2003 Guest - Correspondence added

From:

pyt_inet0 [...] tiscali.fr

[guest - Sun Sep 7 08:54:12 2003]: Looking a little deeper into it, it looks like the TXXX and WXXX frames get sucked into the binary data of the APIC frame, as if the size of the APIC frame was interpreted wrong... Attached is a much smaller MP3 file with just an APIC and a TXXX. Pyt.

Download clean with APIC & TXXX.mp3
audio/mpeg 20.2k

Message body not shown because it is not plain text.

Tue Sep 16 20:16:25 2003 Guest - Correspondence added

From:

pyt_inet0 [...] tiscali.fr

Found the cause of the problem: in v2.4, the length of the APIC frame is encoded as 4 synchsafe bytes (bytes with MSB forced to 0), rather than 4 plain bytes, as in v2.3. Thus the arithmetic to compute the actual length from the encoding is slightly different from that used for version 2.3; this difference is not accounted for in v1.02 of MP3::Info. I'll be mailing a fix. Cheers, Pyt.

Sun Jan 09 00:38:16 2005 cnandor [...] cpan.org - Fixed in 1.10 added

Sun Jan 09 00:38:16 2005 cnandor [...] cpan.org - Status changed from 'new' to 'resolved'