Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Edit->Settings" causes Memory Leak (and thus Crash at Exit) #6910

Closed
JohannesLorenz opened this issue Oct 1, 2023 · 2 comments
Closed

"Edit->Settings" causes Memory Leak (and thus Crash at Exit) #6910

JohannesLorenz opened this issue Oct 1, 2023 · 2 comments

Comments

@JohannesLorenz
Copy link
Contributor

Bug Summary

When you open the SetupDialog (via Edit->Settings), you get a crash

Steps to reproduce

  1. Start LMMS
  2. Open "Edit->Settings" from the menu
  3. Press cancel
  4. Close LMMS (e.g. using the top right "X" of your Window Manager)

Expected behavior

No segfault.

Actual behavior

Segfault, indicating a memory leak. I know that we can just turn the memory leak detection off, but this bug is about the memory leak itself.

The bug seems to be in there for a while, even before #6486 (7aca8ae).

valgrind analysis should help to find the leak. Otherwise, you would need to deconstruct the dialog, bit by bit, and wait until the error vanishes.

Affected LMMS versions

33d1bad (master)

Logs

Click to expand
lmms: /home/johannes/cprogs/lmms/work/src/3rdparty/rpmalloc/rpmalloc/rpmalloc/rpmalloc.c:1328: _rpmalloc_span_finalize: Assertion `(span->list_size == span->used_count) && "Memory leak detected"' failed.
Aborted (core dumped)
@Rossmaxx
Copy link
Contributor

Now that rpmalloc is removed+ a few leak fixes, does this issue still occur @JohannesLorenz ?

@zonkmachine
Copy link
Member

This was fixed with removing of the MemoryManager in c991a85

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants