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.

Sign up for GitHub

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

Jump to bottom

Can't find the first cluster element #17

Open
pitdicker opened this issue Feb 14, 2025 · 3 comments
Open

Can't find the first cluster element #17

pitdicker opened this issue Feb 14, 2025 · 3 comments

Comments

@pitdicker
Copy link

pitdicker commented Feb 14, 2025 •
edited
Loading

I have two files that matroska-demuxer fails to parse with the error message 'can't find the first cluster element'.
This started happening after I made some changes to them with MKVToolNix Gui. mkvalidator thinks the files are valid.
Can you help me figure out what is going wrong?

I have tested with version 0.5 and 0.6.
The error originates in find_first_cluster_offset, in the loop that calls next_element.
I can privately share the video if that helps.
@gwen-lg
Copy link
Contributor

gwen-lg commented Feb 14, 2025 •
edited
Loading

@pitdicker: can you share with me at least one of the file ? (my contact email on my web-site)
I had a problem with the same symptoms, but it's supposed to be fixed in version 0.6 (fixed by fb30bf8)

@pitdicker
Copy link
Author

Thank you, sure! I'll send a mail to the address of your git commits, okay?

@gwen-lg
Copy link
Contributor

gwen-lg commented Feb 14, 2025

I have understood the problem, need some time to think how it's best to fix it.

gwen-lg added a commit to gwen-lg/matroska-demuxer that referenced this issue Feb 14, 2025
@gwen-lg
fix the search the first data cluster ...
c8f4d25 
when track element is located after the cluster segments.
And the first Cluster is not indicated in `SeekHead`.

This should fix issue [Can't find the first cluster element hasenbanck#17](hasenbanck#17)
gwen-lg added a commit to gwen-lg/matroska-demuxer that referenced this issue Feb 14, 2025
@gwen-lg
fix: Start the search the first data cluster from start of Segment El…
27f4fd7 
…ement

Start looking after the `Track Element` is not valid when located after the Cluster segments.

This should fix issue [Can't find the first cluster element hasenbanck#17](hasenbanck#17)
@gwen-lg gwen-lg mentioned this issue Feb 14, 2025
Merged
hasenbanck pushed a commit that referenced this issue Feb 15, 2025
@gwen-lg @hasenbanck
fix: Start the search the first data cluster from start of Segment El…
01aad95 
…ement

Start looking after the `Track Element` is not valid when located after the Cluster segments.

This should fix issue [Can't find the first cluster element #17](#17)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pitdicker @gwen-lg