Closed Bug 886660 Opened 11 years ago Closed 11 years ago

FFTBlock::SetFFTSize() leaks kiss_fftr_cfg objects

Categories

(Core :: Web Audio, defect)

x86_64
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla25
Tracking Status
firefox24 + fixed
firefox25 + fixed

People

(Reporter: karlt, Assigned: ehsan.akhgari)

References

Details

(Keywords: memory-leak, Whiteboard: [blocking-webaudio+][qa-])

Attachments

(1 file)

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
Summary: SetFFTSize() leaks kiss_fftr_cfg objects → FFTBlock::SetFFTSize() leaks kiss_fftr_cfg objects
Attached patch Patch (v1)Splinter Review
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #767409 - Flags: review?(roc)
Whiteboard: [blocking-webaudio+]
https://hg.mozilla.org/mozilla-central/rev/da6afcd4271b
Status: ASSIGNED → RESOLVED
Closed: 11 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.

Attachment

General

Created:
Updated:
Size: