search

saecki / mp4ameta

A library for reading and writing iTunes style MPEG-4 audio metadata
Apache License 2.0
35 stars 5 forks source link

`Parsing: Read length of mdat which is less than 8 bytes: 1` #17

Closed probablykasper closed 3 years ago

probablykasper commented 3 years ago

Hi, I'm getting this error from Tag::read_from_path(): Parsing: Read length of mdat which is less than 8 bytes: 1. iTunes, QuickTime, VLC and music-metadata are able to handle the file just fine.

I tested mp4ameta on 4 of the m4as I have, and this issue turned up, so not sure how common this error would be for the other 4000 m4as I've got.

Can send over the m4a privately if you need it.

saecki commented 3 years ago

Hi thanks for reporting this bug. The file would be very helpful indeed, so if you could send it over that would be great.

probablykasper commented 3 years ago

@Saecki Sent it to your email

saecki commented 3 years ago

Funny thing on the dev branch I was already implementing support for 64bit atom sizes, which are denoted by the length of 1 which would otherwise be illegal. I had not tested it yet, though it seems to work just fine on the file you sent me. Currently only reading is supported but I will merge these changes into master for now.

probablykasper commented 3 years ago

@Saecki Ah that's great! Tried using commit b7753ac4cf7bd744178fd74fefb8d15bf4fcde01 to read, and it worked.