-
-
Notifications
You must be signed in to change notification settings - Fork 395
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
[Bug]: Memory leak with flatpak upgrade #5816
Comments
Note: it doesn't happen with plain Also I believe it might be related to the code path going through EOL runtimes. |
Valgrind also show it
Note the amount of "indirect" |
hfiguiere
added
bug
cli
Issues involving the flatpak command
and removed
needs triage
labels
May 22, 2024
hfiguiere
added a commit
to hfiguiere/flatpak
that referenced
this issue
May 24, 2024
Close flatpak#5816 Also reenable leak checking in the test Signed-off-by: Hubert Figuière <hub@figuiere.net>
hfiguiere
added a commit
to hfiguiere/flatpak
that referenced
this issue
May 24, 2024
Close flatpak#5816 Also reenable some leak checking in CI Signed-off-by: Hubert Figuière <hub@figuiere.net>
hfiguiere
added a commit
to hfiguiere/flatpak
that referenced
this issue
May 24, 2024
Close flatpak#5816 Signed-off-by: Hubert Figuière <hub@figuiere.net>
hfiguiere
added a commit
to hfiguiere/flatpak
that referenced
this issue
May 24, 2024
Close flatpak#5816 Signed-off-by: Hubert Figuière <hub@figuiere.net>
hfiguiere
changed the title
[Bug]: Memory leak with flaptak upgrade
[Bug]: Memory leak with flatpak upgrade
May 24, 2024
TingPing
pushed a commit
that referenced
this issue
May 24, 2024
Close #5816 Signed-off-by: Hubert Figuière <hub@figuiere.net>
hfiguiere
added a commit
to hfiguiere/flatpak
that referenced
this issue
Jun 18, 2024
Close flatpak#5816 Also reenable leak checking in the test Signed-off-by: Hubert Figuière <hub@figuiere.net>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Checklist
Flatpak version
master
What Linux distribution are you using?
Fedora Silverblue
Linux distribution version
39
What architecture are you using?
x86_64
How to reproduce
Run
flatpak upgrade
(flatpak is built locally with ASan)Then there is a long list of indirect leaks.
Expected Behavior
No memory leak
Actual Behavior
ASan show memory leaks.
Additional Information
I have looked quickly and can't find how it leaks. The dir gets directly added to a
g_ptr_array()
and this should be alright disposing of it.The text was updated successfully, but these errors were encountered: