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 683770 - Fix some memory leaks
Fix some memory leaks
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gvfs-maint
gvfs-maint
Depends on:
Blocks:
 
 
Reported: 2012-09-11 05:18 UTC by William Jon McCann
Modified: 2012-09-11 12:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix some memory leaks (3.96 KB, patch)
2012-09-11 05:18 UTC, William Jon McCann
committed Details | Review

Description William Jon McCann 2012-09-11 05:18:37 UTC
See patch.
Comment 1 William Jon McCann 2012-09-11 05:18:38 UTC
Created attachment 223983 [details] [review]
Fix some memory leaks
Comment 2 Tomas Bzatek 2012-09-11 12:21:04 UTC
Review of attachment 223983 [details] [review]:

Looks good, thanks for spotting these mistakes.

::: client/gvfsdaemondbus.c
@@ +264,2 @@
 typedef struct {
+  char *dbus_id;

I wonder why I didn't get any compiler warning for this.
Comment 3 William Jon McCann 2012-09-11 12:37:43 UTC
Attachment 223983 [details] pushed as 2dbfccc - Fix some memory leaks