Merge lp:~ubuntu-multiseat/lightdm/safe-login1-signal-subscribe into lp:lightdm

[Laércio de Sousa]

Make g_dbus_connection_signal_subscribe() call in login1.c a little bit safer (really?)

TBH I'm not sure this change is really needed or safer, but, for some reason, this is the way signal subscription is done in GDM [1], so I'm proposing it here just in case.

[1] https://git.gnome.org/browse/gdm/tree/daemon/gdm-local-display-factory.c

[Robert Ancell (robert-ancell)]

It shouldn't be required since we unsubscribe the signal when we finalize the object but it can't hurt.

review: Approve

=== modified file 'src/login1.c'
--- src/login1.c	2014-08-22 01:56:45 +0000
+++ src/login1.c	2014-08-26 18:26:51 +0000
@@ -348,8 +348,8 @@
                                                                    NULL,
                                                                    G_DBUS_SIGNAL_FLAGS_NONE,
                                                                    signal_cb,
-                                                                   service,
-                                                                   NULL);
+                                                                   g_object_ref (service),
+                                                                   g_object_unref);
 
     result = g_dbus_connection_call_sync (service->priv->connection,
                                           LOGIN1_SERVICE_NAME,