Mixxx is leaking memory

Bug #1369051 reported by Tuukka Pasanen on 2014-09-13
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mixxx
Critical
Tuukka Pasanen

Bug Description

Mixxx is leaking heavily in memory. See:
http://pastebin.com/yMPvpz8R

Tuukka Pasanen (pasanen-tuukka) wrote :
Changed in mixxx:
assignee: nobody → Tuukka Pasanen (pasanen-tuukka)
status: New → In Progress
Daniel Schürmann (daschuer) wrote :

Bug #1335823 might be related

Changed in mixxx:
milestone: none → 1.12.0
importance: Undecided → Critical
Tuukka Pasanen (pasanen-tuukka) wrote :

Yes bug #1335823 probably is related. I spotted at least one Reverb related bad_alloc/free issue. Currently I have pushed memory leaks from 22 MB to 2 MB. Easy ones are done and now I've got to check why Rubberband and Vamp leaks also. Is it our fault or library issue. Rest of the leaks are under 1k so they are trivial but i'll try to crush all that I can.

Curren git is located https://github.com/illuusio/mixxx/tree/leak-fix

Daniel Schürmann (daschuer) wrote :

Thank you very much!

Would you mind to do a pull request with the current state?
You fixes are looking self containing.

RJ Ryan (rryan) wrote :

Can we mark this fixed?

Tuukka Pasanen (pasanen-tuukka) wrote :

More or less.. I think so.. Leaks are there but they don't get on the way so just close it..

Daniel Schürmann (daschuer) wrote :

I would like to keep it open as a kind of meta bug since it is not completely solved.

RJ Ryan (rryan) wrote :

IMO that sort of bug is not very useful.

It will occupy a row on our milestone page that is never fixed -- so I'm removing it from 1.12.0.

This is on the same level as "Mixxx shouldn't dereference NULL pointers" or "Mixxx should have high test coverage" -- there isn't a clear metric for the impact or scope of the bug.

Changed in mixxx:
milestone: 1.12.0 → none
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers