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

Deadlock on incorrect QuickTime file #111

Closed
StudioMaX opened this issue Jun 11, 2017 · 2 comments
Closed

Deadlock on incorrect QuickTime file #111

StudioMaX opened this issue Jun 11, 2017 · 2 comments

Comments

@StudioMaX
Copy link
Collaborator

StudioMaX commented Jun 11, 2017

File:

deadf.zip

Probably related issues:

https://trac.ffmpeg.org/ticket/3866
http://www.getid3.org/phpBB3/viewtopic.php?f=7&t=836
http://www.getid3.org/phpBB3/viewtopic.php?t=1813

Description:

After debugging a while I found that this file contain incorrect value in number_entries of the Atom stsd - for some reason this file has 08 00 00 01 instead of 00 00 00 01.

I cannot play this file on my PC. But because of this error, the parsing of this file is looped to infinite.

StudioMaX added a commit to StudioMaX/getID3 that referenced this issue Jun 11, 2017
StudioMaX added a commit to StudioMaX/getID3 that referenced this issue Jun 11, 2017
@StudioMaX
Copy link
Collaborator Author

Proposed fix: #112

StudioMaX added a commit to StudioMaX/getID3 that referenced this issue Jun 11, 2017
JamesHeinrich added a commit that referenced this issue Jun 11, 2017
If an incorrect value is set in stsd.number_entries then the script can
appear to hang (not infinite, but very long loop). Detect, provide
warning, override as necessary.
@JamesHeinrich
Copy link
Owner

Fixed in 335f544

StudioMaX pushed a commit to StudioMaX/getID3 that referenced this issue May 26, 2019
If an incorrect value is set in stsd.number_entries then the script can
appear to hang (not infinite, but very long loop). Detect, provide
warning, override as necessary.
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

2 participants