Mixxx is leaking memory

Bug #1369051 reported by Tuukka Pasanen
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Critical
Tuukka Pasanen

Bug Description

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

Tags: leak
Revision history for this message
Tuukka Pasanen (pasanen-tuukka) wrote :
Changed in mixxx:
assignee: nobody → Tuukka Pasanen (pasanen-tuukka)
status: New → In Progress
Revision history for this message
Daniel Schürmann (daschuer) wrote :

Bug #1335823 might be related

Changed in mixxx:
milestone: none → 1.12.0
importance: Undecided → Critical
Revision history for this message
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

Revision history for this message
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.

Revision history for this message
Tuukka Pasanen (pasanen-tuukka) wrote :
Revision history for this message
Tuukka Pasanen (pasanen-tuukka) wrote :
Revision history for this message
Tuukka Pasanen (pasanen-tuukka) wrote :
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Can we mark this fixed?

Revision history for this message
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..

Revision history for this message
Daniel Schürmann (daschuer) wrote :

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

Revision history for this message
RJ Skerry-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
Revision history for this message
Daniel Schürmann (daschuer) wrote :

We had no reports lately about leaking memory, so we can close this.

Changed in mixxx:
status: In Progress → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7578

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.