Philipp Wolfer

Results 252 comments of Philipp Wolfer

Also I think this issue is basically a duplicate of #207, which is about adding RIFF INFO support to mutagen

If it gets officially exposed in some ways IMHO it should be done with Base16 representation

I see. I think the proper way to handle this is storing APIC last then, but don't reorder the APIC frames themselves. > I'd prefer the other clients getting fixed,...

Looking at the file I notice there is a mismatch in sizes in the LIST INFO chunk. The LIST INFO chunk reports a size of 124 bytes, It contains two...

Now that I wrote this I see what's actually wrong: The file seems to be truncated. It is 75391073 bytes long, but the data sizes inside sum up to 75391144...

> 1. If the contained content is valid (and not truncated) we could somehow recover (we do this in flac where programs used to write wrong sizes a lot) In...

Just to be complete here and share my analysis: I verified the broken file using the script at https://gist.github.com/phw/07f2828cb1e04ba04f908b1473b07aac This outputs the structure of the WAVE file with offsets and...

This central patch also resolves the issue in all AIFF, WAVE and DSDIFF.

@lazka Could you review this? You could check with the file from https://github.com/quodlibet/mutagen/issues/496 Files which are smaller then their internally reported size seem to be quite common. That's both true...

@lazka may I ask for your attention on this issue once again?