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

Regression: error decoding metadata from some m4a files in 0.8.6 #16

Open
mvmn opened this issue Apr 7, 2019 · 0 comments
Open

Regression: error decoding metadata from some m4a files in 0.8.6 #16

mvmn opened this issue Apr 7, 2019 · 0 comments

Comments

@mvmn
Copy link

mvmn commented Apr 7, 2019

I've tried to play some m4a files I've bought from iTunes store, and everything worked with version 0.8.5 from Maven. But when I started to work with current code from master - MP4Container creation failed with this error:

Exception in thread "main" java.io.IOException: error while decoding box '����' at offset 73633: box too large for parent
	at net.sourceforge.jaad.mp4.boxes.BoxFactory.parseBox(BoxFactory.java:347)
	at net.sourceforge.jaad.mp4.boxes.BoxImpl.readChildren(BoxImpl.java:110)
	at net.sourceforge.jaad.mp4.boxes.impl.MetaBox.decode(MetaBox.java:26)
	at net.sourceforge.jaad.mp4.boxes.BoxFactory.parseBox(BoxFactory.java:353)
	at net.sourceforge.jaad.mp4.boxes.BoxImpl.readChildren(BoxImpl.java:110)
	at net.sourceforge.jaad.mp4.boxes.BoxFactory.parseBox(BoxFactory.java:357)
	at net.sourceforge.jaad.mp4.boxes.BoxImpl.readChildren(BoxImpl.java:110)
	at net.sourceforge.jaad.mp4.boxes.BoxFactory.parseBox(BoxFactory.java:357)
	at net.sourceforge.jaad.mp4.MP4Container.readContent(MP4Container.java:90)
	at net.sourceforge.jaad.mp4.MP4Container.<init>(MP4Container.java:81)
	at net.sourceforge.jaad.Play.decodeMP4(Play.java:63)
	at net.sourceforge.jaad.Play.main(Play.java:49)

Checking differences between 0.8.5 and 0.8.6 I've found this change:
4c9fce6
When commented out, decoding works again.

I'm sorry I can't provide more relevant data and can't attach m4a file (because that'd be a copyright violation). If needed I suppose though I could provide some portion of the file content, e.g. where headers are.

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

No branches or pull requests

1 participant