Closed
Bug 867089
Opened 11 years ago
Closed 11 years ago
WebAudio divide-by-zero crash [mozilla::dom::AudioBufferSourceNodeEngine::Resampler]
Categories
(Core :: Web Audio, defect)
Core
Web Audio
Tracking
()
RESOLVED
FIXED
mozilla23
People
(Reporter: posidron, Assigned: padenot)
References
Details
(Keywords: crash, testcase)
Crash Data
Attachments
(4 files, 1 obsolete file)
11.22 KB,
text/html
|
Details | |
4.24 KB,
text/plain
|
Details | |
4.25 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
2.33 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
content/media/webaudio/AudioBufferSourceNode.cpp:129 mResampler = speex_resampler_init(mChannels, mSampleRate, * ComputeFinalOutSampleRate(), SPEEX_RESAMPLER_QUALITY_DEFAULT, nullptr); Tested with m-i changeset: 130174:ea5490a3bca7
Reporter | ||
Comment 1•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → paul
Comment 2•11 years ago
|
||
On Windows: bp-6d206d97-c7b5-4ffd-ae17-2ae212130430.
Assignee: paul → nobody
Crash Signature: [@ mozalloc_abort(char const*) | NS_DebugBreak | mozilla::dom::AudioBufferSourceNodeEngine::Resampler(unsigned int)]
[@ update_filter ]
OS: Mac OS X → All
Hardware: x86_64 → All
Updated•11 years ago
|
Assignee: nobody → paul
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•11 years ago
|
||
Sorry, forgot to `hg add`.
Attachment #743585 -
Flags: review?(ehsan)
Assignee | ||
Updated•11 years ago
|
Attachment #743577 -
Attachment is obsolete: true
Attachment #743577 -
Flags: review?(ehsan)
Comment 6•11 years ago
|
||
Comment on attachment 743562 [details] [diff] [review] Validate the playbackRate before using it. Review of attachment 743562 [details] [diff] [review]: ----------------------------------------------------------------- r=me minus the cubeb changes which don't seem intentional. ::: content/media/webaudio/AudioBufferSourceNode.cpp @@ +11,5 @@ > #include "AudioNodeStream.h" > #include "AudioDestinationNode.h" > #include "PannerNode.h" > #include "speex/speex_resampler.h" > +#include "WebAudioUtils.h" You shouldn't need this #include here.
Attachment #743562 -
Flags: review?(ehsan) → review+
Comment 7•11 years ago
|
||
Comment on attachment 743585 [details] [diff] [review] Test. Review of attachment 743585 [details] [diff] [review]: ----------------------------------------------------------------- ::: content/media/webaudio/test/test_bug867089.html @@ +27,5 @@ > + source2.playbackRate.value = -1.0; > + source2.connect(ctx.destination); > + source2.start(0); > + > + var source2 = ctx.createBufferSource(); Nit: source3.
Attachment #743585 -
Flags: review?(ehsan) → review+
Comment 8•11 years ago
|
||
Actually, you're not waiting for the graph to play back in the test, so it may never get enough time to render before being paused when we go and run the next test. Does this test crash without your patch?
Assignee | ||
Comment 9•11 years ago
|
||
Yes, consistently. Should I put a |setTimeout| before finishing, for good measure?
Comment 10•11 years ago
|
||
(In reply to comment #9) > Yes, consistently. Should I put a |setTimeout| before finishing, for good > measure? Hmm, no, if the test crashes without your patches, then it's good!
Assignee | ||
Comment 11•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ab9c9d0819a2 https://hg.mozilla.org/integration/mozilla-inbound/rev/5447d49a2c6f
Comment 12•11 years ago
|
||
Backed out for test failures. https://hg.mozilla.org/integration/mozilla-inbound/rev/4b68073547c0 https://tbpl.mozilla.org/php/getParsedLog.php?id=22414207&tree=Mozilla-Inbound
Comment 13•11 years ago
|
||
That's just a typo in the test, will reland now.
Comment 14•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/79220f949406 https://hg.mozilla.org/integration/mozilla-inbound/rev/a0d7d78facd4
Comment 15•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/79220f949406 https://hg.mozilla.org/mozilla-central/rev/a0d7d78facd4
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Comment 16•11 years ago
|
||
Mass moving Web Audio bugs to the Web Audio component. Filter on duckityduck.
Component: Video/Audio → Web Audio
You need to log in
before you can comment on or make changes to this bug.
Description
•