Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ID3: Improper extended header flags are detected but not corrected #21

Closed
lazka opened this issue Jul 4, 2014 · 4 comments
Closed

ID3: Improper extended header flags are detected but not corrected #21

lazka opened this issue Jul 4, 2014 · 4 comments

Comments

@lazka
Copy link
Member

lazka commented Jul 4, 2014

Originally reported by: Christoph Reiter (Bitbucket: lazka, GitHub: lazka)


From korn...@gmail.com on August 31, 2009 07:36:44

mutagen-inspect says:
- MPEG 1 layer 3, 256000 bps, 44100 Hz, 298.89 seconds (audio/mp3)



mplayer says:

Audio only file format detected.
Clip info:
 Title: Punk To Funk
 Artist: FatBoy Slim
 Album: Better Living Through Chemistr
 Year: 1996
 Comment: 
 Track: 9
 Genre: Unknown
==========================================================================
Opening audio decoder: [mp3lib] MPEG layer-2, layer-3
AUDIO: 44100 Hz, 2 ch, s16le, 256.0 kbit/18.14% (ratio: 32000->176400)


first 32Kb of the file are attached

Attachment: 32k_of_failed_file.mp3

Original issue: http://code.google.com/p/mutagen/issues/detail?id=21


@lazka
Copy link
Member Author

lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From joe.wreschnig@gmail.com on September 06, 2009 23:33:01

These files have a corrupt flag in them; the unreleased version of Mutagen has some
code to handle corruption of that flag because it's common, but it amounted to just
skipping over the file.

Such files should now have a much better chance of loading.

Summary: ID3: Improper extended header flags are detected but not corrected

@lazka
Copy link
Member Author

lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From joe.wreschnig@gmail.com on September 06, 2009 23:29:37

This issue was closed by revision r53 .

Status: Fixed

@lazka
Copy link
Member Author

lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From joe.wreschnig@gmail.com on September 07, 2009 11:15:39

Unfortunately the file offers no clues as to its origin. The only unique thing in the
file is PRIV tags for PeakValue and AverageLevel, which I've also never seen before
(foobar uses TXXX frames, iTunes uses COMM frames, and the standard uses RVA2 frames).

If you check out the latest version of Mutagen it should be a drop-in replacement for
whatever you're using. We take API compatibility very seriously.

@lazka
Copy link
Member Author

lazka commented Jul 4, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


From korn...@gmail.com on September 07, 2009 09:10:53

I have a whole bunch of such files, not only mp3's but some flac's as well. Is there
any way to fix the flag in existing files? Any small utility that can be scripted and
run aver the whole media library?  Would be nice to be able to use these files before
the updated version of mutagen makes it into the application dependent on it. Is
there an information what software (converter, tag editor?) causes the corrupt flag,
so I can avoid using it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant