Closed Bug 1181215 Opened 6 years ago Closed 6 years ago
Int32(k Key Track ID, &m Track Id)) failed in MPEG4Extractor .cpp
A/MPEG4Extractor(34284): /builds/slave/m-cen-l64-asan-d-ntly-00000000/build/src/media/libstagefright/frameworks/av/media/libstagefright/MPEG4Extractor.cpp:2649 CHECK(format->findInt32(kKeyTrackID, &mTrackId)) failed.
What is the aim of your mp4 tests? What do you expect the code to do?
Hi Jean-Yves, I am part of the platform fuzzing team. The aim of our team is to fuzz different parts of the code base to find stability and security issues. In the context of media playback a 'pass' would be the browser not crashing/asserting when given a fuzzed file for playback. If you would like more information about fuzzing please feel free to email me or ping me on IRC (:tsmith)
Summary: CHECK(format->findInt32(kKeyTrackID, &mTrackId)) failed in MPEG4Extractor.cpp → Stagefright: CHECK(format->findInt32(kKeyTrackID, &mTrackId)) failed in MPEG4Extractor.cpp
Gerald, here's another Stagefright bug.
Assignee: nobody → gsquelart
Priority: -- → P1
Part 1: Added gtest case.
Attachment #8683614 - Flags: review?(giles)
Part 2: Better track verification. Copied some tests from MPEG4Source constructor to MPEG4Extractor::verifyTrack so that we don't get late surprises, in particular a missing track ID.
Attachment #8683615 - Flags: review?(giles)
The patch in this bug includes a fix for bug 1181213 (unexpected configurationVersion).
Comment on attachment 8683614 [details] [diff] [review] 1181215-gtest.patch Review of attachment 8683614 [details] [diff] [review]: ----------------------------------------------------------------- Thanks for doing all these tests.
Attachment #8683614 - Flags: review?(giles) → review+
Clarified check-in comment, no actual changes; carrying r+.
Check-in in this order: bug 1181719, bug 1181223, bug 1181220, bug 1181215, bug 1181213 try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=20e7b755a695
Whiteboard: [fuzzblocker:stagefright] → [fuzzblocker:stagefright] checkin after 1181719, 1181223, 1181220 but before 1181213
You need to log in before you can comment on or make changes to this bug.