subtract overflow when running dom/media/test/test_eme_non_mse_fails.html

RESOLVED FIXED in Firefox 54

Status

()

Core
Audio/Video: Playback
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: alfredo, Assigned: alfredo)

Tracking

unspecified
mozilla54
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

a year ago
In test dom/media/test/test_eme_non_mse_fails.html, it crashs due to subtract overflow.

thread '<unnamed>' panicked at 'attempt to subtract with overflow', /Users/alfredo/mozilla/mozilla-unified/media/libstagefright/binding/mp4parse_capi/src/lib.rs:677
note: Run with `RUST_BACKTRACE=1` for a backtrace.
Redirecting call to abort() to mozalloc_abort

Hit MOZ_CRASH() at /Users/alfredo/mozilla/mozilla-unified/memory/mozalloc/mozalloc_abort.cpp:33
Process 98393 stopped
* thread #82: tid = 0xf53c9f, 0x000000010000e2b1 libmozglue.dylib`mozalloc_abort(msg="Redirecting call to abort() to mozalloc_abort\n") + 97 at mozalloc_abort.cpp:33, name = 'MediaPD~oder #1', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x000000010000e2b1 libmozglue.dylib`mozalloc_abort(msg="Redirecting call to abort() to mozalloc_abort\n") + 97 at mozalloc_abort.cpp:33
   30  	#ifdef MOZ_WIDGET_ANDROID
   31  	    abortThroughJava(msg);
   32  	#endif
-> 33  	    MOZ_CRASH();
   34  	}
   35
(Assignee)

Updated

a year ago
Assignee: nobody → ayang
Blocks: 1161350
Hooray for overflow checking.
Blocks: 1342848
Duplicate of this bug: 1342848
Comment hidden (mozreview-request)

Comment 5

a year ago
mozreview-review
Comment on attachment 8842339 [details]
Bug 1342852 - update rust mp4 parser.

https://reviewboard.mozilla.org/r/116220/#review117986
Attachment #8842339 - Flags: review?(kinetik) → review+

Comment 6

a year ago
Pushed by ayang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6085498051cc
update rust mp4 parser. r=kinetik

Comment 7

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6085498051cc
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.