After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 778804 - Plug some memory leaks
Plug some memory leaks
Status: RESOLVED FIXED
Product: gnome-autoar
Classification: Other
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GNOME Autoar maintainer(s)
GNOME Autoar maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2017-02-17 02:41 UTC by Matthias Clasen
Modified: 2021-06-14 14:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Plus some memory leaks (1.35 KB, patch)
2017-02-17 02:41 UTC, Matthias Clasen
committed Details | Review

Description Matthias Clasen 2017-02-17 02:41:52 UTC
valgrind complained that the g_list_copy_deep call here leaks
memory. In fact, the set_property implementation already takes
care of copying the list and reffing the objects, so don't
do it here.
Comment 1 Matthias Clasen 2017-02-17 02:41:55 UTC
Created attachment 346037 [details] [review]
Plus some memory leaks
Comment 2 Carlos Soriano 2017-03-27 21:31:35 UTC
Review of attachment 346037 [details] [review]:

Thanks Mathias
Comment 3 André Klapper 2020-11-15 15:02:31 UTC
Patch is still awaiting merge. Would someone please do so? TIA.
Comment 4 André Klapper 2021-06-10 11:33:11 UTC
Ting-Wei Lan, Carlos: Ping - could someone review / merge this two-line patch? TIA
Comment 5 Ondrej Holy 2021-06-14 14:48:33 UTC
Comment on attachment 346037 [details] [review]
Plus some memory leaks

commit a6de6e3dfdc67d7302bffc4c1366ce0d7d507dd2