Skip to content

EbmlMaster: fix leak on upper element found inside the last element#337

Open
robUx4 wants to merge 1 commit into
Matroska-Org:masterfrom
robUx4:leak-upper
Open

EbmlMaster: fix leak on upper element found inside the last element#337
robUx4 wants to merge 1 commit into
Matroska-Org:masterfrom
robUx4:leak-upper

Conversation

@robUx4

@robUx4 robUx4 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

We found an upper element and there is not data to read after the current element within its parent. That upper element is misplaced and should be discarded.
The last element is truncated and doesn't contain the rest of its data.

We found an upper element and there is not data to read after the current
element within its parent. That upper element is misplaced and should be
discarded.
The last element is truncated and doesn't contain the rest of its data.
@robUx4 robUx4 added the bug label Jun 2, 2026
@mbunkus

mbunkus commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

@robUx4 I just noticed that there are still two of your PRs on master pending that do pretty much the same thing? I think? This one & #321, the branches aptly called leak-upper & upper-leak. As they conflict, can you please take a look & decide which one to keep? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants