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

Decode AAC return error when AU data length is zero #544

Closed
wants to merge 1 commit into from

Conversation

BenLocal
Copy link
Contributor

try fix #542

@aler9
Copy link
Member

aler9 commented May 5, 2024

Hello, although a blank access unit is surely an invalid unit, the RTP decoder is not in charge of validating incoming access units. Checks inside the RTP decoder are meant to avoid panics and OOM situations. Adding additional checks would decrease performance and would not be enough anyway to guarantee the validity of the unit.

Checks on the content of access units are already performed by codec-specific decoders or interpreters, during decoding or parsing.

@aler9 aler9 closed this May 5, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Nov 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decode AAC payload when AU data length is zero
2 participants