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

Ignores faad2 "bitstream value not allowed by specification" errors. #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

carter-thaxton
Copy link
Contributor

These errors resulted in "invalid file format" errors on otherwise good files.
The faad2 decoder simply skips over these packets and moves on, and it's unrelated to any of the functionality of aacgain.

These errors resulted in "invalid file format" errors on otherwise good files.
The faad2 decoder simply skips over these packets and moves on, and it's unrelated to any of the functionality of aacgain.
@dgilman
Copy link
Owner

dgilman commented Jul 13, 2022

Do you have one of these files that you could send me? If you want to do it OOB my email is in the commit logs. I wonder if bumping to the latest faad helps? It's probably worth doing anyway, even if it doesn't fix this issue.

@carter-thaxton
Copy link
Contributor Author

I just sent you an email with some test files.

@dgonyier
Copy link

dgonyier commented Apr 9, 2024

@carter-thaxton does this have to do with AAC streams that have Advanced Audio Codec Low Complexity with Spectral Band Replication ? It seems all files that include Spectral Band Replication are reported as invalid file formats.

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

Successfully merging this pull request may close these issues.

3 participants