chiark / gitweb /
timedated: do not incorrectly close non-opened dbus connection
authorShawn Landden <shawnlandden@gmail.com>
Mon, 3 Dec 2012 00:50:55 +0000 (00:50 +0000)
committerKay Sievers <kay@vrfy.org>
Mon, 3 Dec 2012 00:56:42 +0000 (01:56 +0100)
Fix the fallowing error when no system dbus available:

Failed to get system D-Bus connection: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
process 14920: arguments to dbus_connection_close() were incorrect, assertion "connection != NULL" failed in file ../../dbus/dbus-connection.c line 2889.
This is normally a bug in some application using the D-Bus library.
process 14920: arguments to dbus_connection_unref() were incorrect, assertion "connection != NULL" failed in file ../../dbus/dbus-connection.c line 2776.
This is normally a bug in some application using the D-Bus library.

src/timedate/timedated.c

index 40ba25509059036ee359839da07a4da36f8ecd70..784dadc631bde15e49c6b7aab9889d172a241e55 100644 (file)
@@ -936,7 +936,7 @@ static int connect_bus(DBusConnection **_bus) {
         if (!bus) {
                 log_error("Failed to get system D-Bus connection: %s", bus_error_message(&error));
                 r = -ECONNREFUSED;
-                goto fail;
+                goto fail2;
         }
 
         dbus_connection_set_exit_on_disconnect(bus, FALSE);
@@ -968,7 +968,7 @@ static int connect_bus(DBusConnection **_bus) {
 fail:
         dbus_connection_close(bus);
         dbus_connection_unref(bus);
-
+fail2:
         dbus_error_free(&error);
 
         return r;