fix issues found by coverty analyser

diff --git a/src/daemon/dbus/gsignond-dbus-server.c b/src/daemon/dbus/gsignond-dbus-server.c
index e948a0f..f6996f3 100644 (file)
--- a/src/daemon/dbus/gsignond-dbus-server.c
+++ b/src/daemon/dbus/gsignond-dbus-server.c
@@ -350,8 +350,11 @@ GSignondDbusServer * gsignond_dbus_server_new_with_address (const gchar *address
 
     g_dbus_server_start (server->priv->bus_server);
 
-    if (file_path)
-        g_chmod (file_path, S_IRUSR | S_IWUSR);
+    if (file_path) {
+        if (g_chmod (file_path, S_IRUSR | S_IWUSR) < 0) {
+            WARN ("Failed to set socket permissions : %s", strerror(errno));
+        }
+    }
 
     return server;
 }

diff --git a/src/daemon/dbus/gsignond-dbus-signonui-adapter.c b/src/daemon/dbus/gsignond-dbus-signonui-adapter.c
index a4392cd..cbbf065 100644 (file)
--- a/src/daemon/dbus/gsignond-dbus-signonui-adapter.c
+++ b/src/daemon/dbus/gsignond-dbus-signonui-adapter.c
@@ -179,7 +179,7 @@ _setup_ui_connection (GSignondDbusSignonuiAdapter *adapter)
                 SIGNONUI_IFACE, "getBusAddress", g_variant_new ("()"), 
                 G_VARIANT_TYPE_TUPLE, G_DBUS_CALL_FLAGS_NONE, -1, NULL, &err);
     if (!reply) {
-        WARN ("Failed to get signon ui bus address : %s", err->message);
+        WARN ("Failed to get signon ui bus address : %s", err ? err->message : "unknown");
         goto fail;
     }