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 711768 - Fix memory leaks in libgmodule tests
Fix memory leaks in libgmodule tests
Status: RESOLVED OBSOLETE
Product: glib
Classification: Platform
Component: gmodule
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks: 711767
 
 
Reported: 2013-11-10 10:21 UTC by Stef Walter
Modified: 2018-05-24 15:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
module-test: Fix leaks in test (697 bytes, patch)
2013-11-10 13:38 UTC, Stef Walter
committed Details | Review

Description Stef Walter 2013-11-10 10:21:04 UTC
In order to make libgmodule tests be valgrindable, I'll post patches
which fix memory leaks.

In general it's good practice to free stuff in tests, so that we can run
through those code paths, and make sure we received appropriate
references/memory.
Comment 1 Stef Walter 2013-11-10 13:38:28 UTC
Created attachment 259420 [details] [review]
module-test: Fix leaks in test
Comment 2 Colin Walters 2013-11-10 15:09:36 UTC
Review of attachment 259420 [details] [review]:

Looks right.
Comment 3 Stef Walter 2013-11-10 21:54:30 UTC
Comment on attachment 259420 [details] [review]
module-test: Fix leaks in test

Attachment 259420 [details] pushed as 6294d73 - module-test: Fix leaks in test
Comment 4 Stef Walter 2013-11-11 07:49:57 UTC
Leaving this open for now as I think there may be a few more tests I haven't found yet.
Comment 5 GNOME Infrastructure Team 2018-05-24 15:52:41 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/glib/issues/781.