Closed
Bug 878014
Opened 11 years ago
Closed 11 years ago
WebAudio Assertion failure: readPosition >= 0.0 (Why are we reading before the beginning of the buffer?) [@mozilla::dom::DelayNodeEngine::ProduceAudioBlock]
Categories
(Core :: Web Audio, defect)
Tracking
()
RESOLVED
FIXED
mozilla24
| Tracking | Status | |
|---|---|---|
| firefox21 | --- | unaffected |
| firefox22 | --- | disabled |
| firefox23 | --- | disabled |
| firefox24 | --- | fixed |
| firefox-esr17 | --- | unaffected |
| b2g18 | --- | unaffected |
People
(Reporter: posidron, Assigned: ehsan.akhgari)
References
Details
(4 keywords, Whiteboard: [adv-main24-])
Attachments
(3 files)
The testcase will crash with an assertion failure in a debug build. In a non debug build, the testcase exposes the same crash as in bug 877523 for which a patch has been checked in with rev e12be295c0b1.
content/media/webaudio/DelayNode.cpp:215
// Write the input sample to the correct location in our buffer
if (input) {
* buffer[writeIndex] = input[i] * aInput.mVolume;
}
Tested with http://hg.mozilla.org/integration/mozilla-inbound/rev/db7a405f0a45
|
Reporter | |
Comment 1•11 years ago
|
||
|
Assignee | |
Comment 2•11 years ago
|
||
(In reply to Christoph Diehl [:cdiehl] from comment #0)
> The testcase will crash with an assertion failure in a debug build. In a non
> debug build, the testcase exposes the same crash as in bug 877523 for which
> a patch has been checked in with rev e12be295c0b1.
Sure, this bug is very different than bug 877523. And FWIW after I fix this there may still be more bugs in this code! ;-)
|
|
Assignee | |
Comment 3•11 years ago
|
||
Attachment #756710 -
Flags: review?(roc) → review+
|
|
Assignee | |
Comment 4•11 years ago
|
||
|
||
Comment 5•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
status-firefox24:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
|
|
Assignee | |
Comment 6•11 years ago
|
||
Mass moving Web Audio bugs to the Web Audio component. Filter on duckityduck.
Component: Video/Audio → Web Audio
|
||
Updated•11 years ago
|
status-firefox21:
--- → unaffected
status-firefox22:
--- → disabled
status-firefox23:
--- → affected
status-firefox-esr17:
--- → unaffected
|
||
Updated•11 years ago
|
|
||
Comment 7•11 years ago
|
||
Test disabled on B2G due to frequent timeouts.
https://hg.mozilla.org/integration/mozilla-inbound/rev/9df04b16a655
|
|
||
Updated•11 years ago
|
Whiteboard: [adv-main24-]
|
||
Updated•11 years ago
|
status-b2g18:
--- → unaffected
|
|
||
Updated•11 years ago
|
Group: core-security
You need to log in
before you can comment on or make changes to this bug.
Description
•