WebAudio Assertion failure: mWriteIndex <= bufferSize [@mozilla::dom::AnalyserNode::AppendChunk]

RESOLVED FIXED in mozilla24

Status

()

Core
Web Audio
--
critical
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: posidron, Assigned: Ehsan)

Tracking

(Blocks: 1 bug, {assertion, crash, testcase})

Trunk
mozilla24
x86_64
Mac OS X
assertion, crash, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
Created attachment 758991 [details]
testcase
(Reporter)

Comment 1

5 years ago
Created attachment 758992 [details]
callstack
(Reporter)

Comment 2

5 years ago
Created attachment 758993 [details]
testcase

reduced testcase
Attachment #758991 - Attachment is obsolete: true
(Assignee)

Comment 3

5 years ago
Created attachment 759595 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #759595 - Flags: review?(roc)
Comment on attachment 759595 [details] [diff] [review]
Patch (v1)

Review of attachment 759595 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/media/webaudio/AnalyserNode.cpp
@@ +281,5 @@
> +    // Copy a maximum bufferSize samples.
> +    chunkCount = bufferSize;
> +  }
> +
> +  PodCopy(mBuffer.Elements() + mWriteIndex, static_cast<const float*>(aChunk.mChannelData[0]), chunkCount);

chunkCount is a terrible name here. It's not the count of chunks. Call it chunkDuration.
Attachment #759595 - Flags: review?(roc) → review+
https://hg.mozilla.org/mozilla-central/rev/728ac315adcb
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.