Merge lp:~dandrader/geis/mem_leaks into lp:geis

[Daniel d'Andrada]

Plugs some memory leaks.

[Chase Douglas (chasedouglas)]

Looks good :).

review: Approve

=== modified file 'libgeis/backend/grail/geis_grail_backend.c'
--- libgeis/backend/grail/geis_grail_backend.c	2012-09-05 14:59:39 +0000
+++ libgeis/backend/grail/geis_grail_backend.c	2012-09-05 18:42:58 +0000
@@ -1013,6 +1013,9 @@
   geis_register_device(gbe->geis, geis_device, attr_count, attrs);
   _grail_be_subscribe_new_device(gbe, geis_device);
 
+  /* We are not going to hold a pointer to this geis_device ourselves  */
+  geis_device_unref(geis_device);
+
 final_exit:
   return;
 }

=== modified file 'libgeis/backend/grail/geis_grail_xsync.c'
--- libgeis/backend/grail/geis_grail_xsync.c	2012-06-21 21:17:25 +0000
+++ libgeis/backend/grail/geis_grail_xsync.c	2012-09-05 18:42:58 +0000
@@ -112,6 +112,7 @@
 void
 geis_grail_xsync_delete(GeisGrailXSync self)
 {
+  geis_bag_delete(self->alarms);
   free(self);
 }
 

=== modified file 'libgeis/geis.c'
--- libgeis/geis.c	2012-08-07 14:26:11 +0000
+++ libgeis/geis.c	2012-09-05 18:42:58 +0000
@@ -1262,6 +1262,8 @@
     GeisAttr attr = geis_attr_new(GEIS_EVENT_ATTRIBUTE_DEVICE,
                                   GEIS_ATTR_TYPE_POINTER,
                                   device);
+    geis_device_ref(device);
+    geis_attr_set_destructor(attr, (GeisAttrDestructor)geis_device_unref);
 
     geis_event_add_attr(event, attr);
     geis_post_event(geis, event);