mp4-rust: Error: BoxNotFound(hdlr): Not found hdlr

I’m getting this error using videos recoded from phone camera or screen recording -> BoxNotFound(hdlr) This error doesn’t appear when i’m using videos downloaded from social networks or other platfroms.

Ocurs using mp4 = “0.13.0” but read_header works in mp4 = “0.12.0” with all kind of mp4 videos.

let f = std::fs::File::open(temp_file.path()).unwrap();
let size = f.metadata()?.len();
let reader = BufReader::new(f);

let mp4 = mp4::Mp4Reader::read_header(reader, size)?; // -> the error ocurs here

About this issue

Original URL
State: closed
Created a year ago
Comments: 18 (11 by maintainers)

Commits related to this issue

Try to skip extended header in MetaBox. Fixes #102 — committed to w-flo/mp4-rust by w-flo a year ago
Try to skip extended header in MetaBox. Fixes #102 — committed to w-flo/mp4-rust by w-flo a year ago
Try to skip extended header in MetaBox. (#111) Fixes #102 Co-authored-by: Alfred Gutierrez <alfg@users.noreply.github.com> — committed to alfg/mp4-rust by w-flo a year ago

Most upvoted comments

@ajosecueto @mrjackwills It would be interesting to know if #111 fixes this issue for you. I’ve tested the small_bad_mp4.mp4 from this issue thread, and that one appears to be fixed now.

Sorry, I’ve not had proper access to my computer or the internet for a while, but I can confirm that it appears to now work when using the following dependency;

mp4 = { git = "https://github.com/alfg/mp4-rust.git", rev = "55ee392b73801d5aff41dbde36455917e0ac2d19" }

mrjackwills on Jul 27, 2023

@w-flo it works, i tested with some videos and works. No more Not found hdlr.

mp4 = { git = "https://github.com/alfg/mp4-rust.git", rev = "55ee392b73801d5aff41dbde36455917e0ac2d19" }

ajosecueto on Jul 11, 2023

#95 and #111 are merged. Feel free to try again using the latest in master and I will create a release.

mp4 = { git = "https://github.com/alfg/mp4-rust.git", rev = "d6c38642de48360693132fd07552a454016d71c3" }
or
mp4 = { git = "https://github.com/alfg/mp4-rust", branch = "master" }

alfg on Jul 29, 2023

Thanks, all. I’ve been super busy, but I’ll try to make time to review and merge soon. Sorry for the wait!

alfg on Jul 28, 2023

Great! Thanks to both of you for testing. Hopefully this will give the PR a good chance of being merged.

w-flo on Jul 27, 2023

@ajosecueto @mrjackwills It would be interesting to know if #111 fixes this issue for you. I’ve tested the small_bad_mp4.mp4 from this issue thread, and that one appears to be fixed now.

w-flo on Jun 29, 2023

@alfg @w-flo I only tested it against “bad” mp4’s, haven’t looked into the source of the pr fully, but assume “good” mp4’s will still work as before, can test again later

mrjackwills on Jul 28, 2023