FFTBlock::SetFFTSize() leaks kiss_fftr_cfg objects

RESOLVED FIXED in Firefox 24

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: karlt, Assigned: Ehsan)

Tracking

({memory-leak})

unspecified
mozilla25
x86_64
All
Points:
---

Firefox Tracking Flags

(firefox24+ fixed, firefox25+ fixed)

Details

(Whiteboard: [blocking-webaudio+][qa-])

Attachments

(1 attachment)

http://hg.mozilla.org/mozilla-central/rev/22d7a1784228#l1.59

If kiss_fft has leak logging, then it may be possible to test this with
getFloatFrequencyData/getByteFrequencyData() and fftSize on AudioNode in
test_analyserNode.html
(Reporter)

Updated

6 years ago
Summary: SetFFTSize() leaks kiss_fftr_cfg objects → FFTBlock::SetFFTSize() leaks kiss_fftr_cfg objects
(Assignee)

Comment 1

6 years ago
Posted patch Patch (v1)Splinter Review
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #767409 - Flags: review?(roc)
(Assignee)

Updated

6 years ago
Whiteboard: [blocking-webaudio+]
https://hg.mozilla.org/mozilla-central/rev/da6afcd4271b
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Assuming no QA needed here. Please remove [qa-] from the whiteboard and add the verifyme keyword if this needs QA.
Whiteboard: [blocking-webaudio+] → [blocking-webaudio+][qa-]
You need to log in before you can comment on or make changes to this bug.