Merge "Fix crash caused by decryption response delay" into tizen
[platform/upstream/connman.git] / src / dbus.c
old mode 100644 (file)
new mode 100755 (executable)
index 70af617..74b3157
@@ -33,7 +33,7 @@ dbus_bool_t connman_dbus_validate_ident(const char *ident)
 {
        unsigned int i;
 
-       if (ident == NULL)
+       if (!ident)
                return FALSE;
 
        for (i = 0; i < strlen(ident); i++) {
@@ -54,13 +54,13 @@ char *connman_dbus_encode_string(const char *value)
        GString *str;
        unsigned int i, size;
 
-       if (value == NULL)
+       if (!value)
                return NULL;
 
        size = strlen(value);
 
        str = g_string_new(NULL);
-       if (str == NULL)
+       if (!str)
                return NULL;
 
        for (i = 0; i < size; i++) {
@@ -183,11 +183,13 @@ void connman_dbus_property_append_array(DBusMessageIter *iter,
 
        switch (type) {
        case DBUS_TYPE_STRING:
-               variant_sig = DBUS_TYPE_ARRAY_AS_STRING DBUS_TYPE_STRING_AS_STRING;
+               variant_sig = DBUS_TYPE_ARRAY_AS_STRING
+                               DBUS_TYPE_STRING_AS_STRING;
                array_sig = DBUS_TYPE_STRING_AS_STRING;
                break;
        case DBUS_TYPE_OBJECT_PATH:
-               variant_sig = DBUS_TYPE_ARRAY_AS_STRING DBUS_TYPE_OBJECT_PATH_AS_STRING;
+               variant_sig = DBUS_TYPE_ARRAY_AS_STRING
+                               DBUS_TYPE_OBJECT_PATH_AS_STRING;
                array_sig = DBUS_TYPE_OBJECT_PATH_AS_STRING;
                break;
        case DBUS_TYPE_DICT_ENTRY:
@@ -234,19 +236,17 @@ dbus_bool_t connman_dbus_property_changed_basic(const char *path,
        DBusMessage *signal;
        DBusMessageIter iter;
 
-       if (path == NULL)
+       if (!path)
                return FALSE;
 
        signal = dbus_message_new_signal(path, interface, "PropertyChanged");
-       if (signal == NULL)
+       if (!signal)
                return FALSE;
 
        dbus_message_iter_init_append(signal, &iter);
        connman_dbus_property_append_basic(&iter, key, type, val);
 
-       g_dbus_send_message(connection, signal);
-
-       return TRUE;
+       return g_dbus_send_message(connection, signal);
 }
 
 dbus_bool_t connman_dbus_property_changed_dict(const char *path,
@@ -256,19 +256,17 @@ dbus_bool_t connman_dbus_property_changed_dict(const char *path,
        DBusMessage *signal;
        DBusMessageIter iter;
 
-       if (path == NULL)
+       if (!path)
                return FALSE;
 
        signal = dbus_message_new_signal(path, interface, "PropertyChanged");
-       if (signal == NULL)
+       if (!signal)
                return FALSE;
 
        dbus_message_iter_init_append(signal, &iter);
        connman_dbus_property_append_dict(&iter, key, function, user_data);
 
-       g_dbus_send_message(connection, signal);
-
-       return TRUE;
+       return g_dbus_send_message(connection, signal);
 }
 
 dbus_bool_t connman_dbus_property_changed_array(const char *path,
@@ -278,20 +276,18 @@ dbus_bool_t connman_dbus_property_changed_array(const char *path,
        DBusMessage *signal;
        DBusMessageIter iter;
 
-       if (path == NULL)
+       if (!path)
                return FALSE;
 
        signal = dbus_message_new_signal(path, interface, "PropertyChanged");
-       if (signal == NULL)
+       if (!signal)
                return FALSE;
 
        dbus_message_iter_init_append(signal, &iter);
        connman_dbus_property_append_array(&iter, key, type,
                                                function, user_data);
 
-       g_dbus_send_message(connection, signal);
-
-       return TRUE;
+       return g_dbus_send_message(connection, signal);
 }
 
 dbus_bool_t connman_dbus_setting_changed_basic(const char *owner,
@@ -301,13 +297,13 @@ dbus_bool_t connman_dbus_setting_changed_basic(const char *owner,
        DBusMessage *msg;
        DBusMessageIter array, dict;
 
-       if (owner == NULL || path == NULL)
+       if (!owner || !path)
                return FALSE;
 
        msg = dbus_message_new_method_call(owner, path,
                                                CONNMAN_NOTIFICATION_INTERFACE,
                                                "Update");
-       if (msg == NULL)
+       if (!msg)
                return FALSE;
 
        dbus_message_iter_init_append(msg, &array);
@@ -317,9 +313,7 @@ dbus_bool_t connman_dbus_setting_changed_basic(const char *owner,
 
        connman_dbus_dict_close(&array, &dict);
 
-       g_dbus_send_message(connection, msg);
-
-       return TRUE;
+       return g_dbus_send_message(connection, msg);
 }
 
 dbus_bool_t connman_dbus_setting_changed_dict(const char *owner,
@@ -330,13 +324,13 @@ dbus_bool_t connman_dbus_setting_changed_dict(const char *owner,
        DBusMessage *msg;
        DBusMessageIter array, dict;
 
-       if (owner == NULL || path == NULL)
+       if (!owner || !path)
                return FALSE;
 
        msg = dbus_message_new_method_call(owner, path,
                                                CONNMAN_NOTIFICATION_INTERFACE,
                                                "Update");
-       if (msg == NULL)
+       if (!msg)
                return FALSE;
 
        dbus_message_iter_init_append(msg, &array);
@@ -346,9 +340,7 @@ dbus_bool_t connman_dbus_setting_changed_dict(const char *owner,
 
        connman_dbus_dict_close(&array, &dict);
 
-       g_dbus_send_message(connection, msg);
-
-       return TRUE;
+       return g_dbus_send_message(connection, msg);
 }
 
 dbus_bool_t connman_dbus_setting_changed_array(const char *owner,
@@ -359,13 +351,13 @@ dbus_bool_t connman_dbus_setting_changed_array(const char *owner,
        DBusMessage *msg;
        DBusMessageIter array, dict;
 
-       if (owner == NULL || path == NULL)
+       if (!owner || !path)
                return FALSE;
 
        msg = dbus_message_new_method_call(owner, path,
                                                CONNMAN_NOTIFICATION_INTERFACE,
                                                "Update");
-       if (msg == NULL)
+       if (!msg)
                return FALSE;
 
        dbus_message_iter_init_append(msg, &array);
@@ -375,9 +367,7 @@ dbus_bool_t connman_dbus_setting_changed_array(const char *owner,
 
        connman_dbus_dict_close(&array, &dict);
 
-       g_dbus_send_message(connection, msg);
-
-       return TRUE;
+       return g_dbus_send_message(connection, msg);
 }
 
 dbus_bool_t __connman_dbus_append_objpath_dict_array(DBusMessage *msg,
@@ -385,7 +375,7 @@ dbus_bool_t __connman_dbus_append_objpath_dict_array(DBusMessage *msg,
 {
        DBusMessageIter iter, array;
 
-       if (msg == NULL || function == NULL)
+       if (!msg || !function)
                return FALSE;
 
        dbus_message_iter_init_append(msg, &iter);
@@ -406,11 +396,122 @@ dbus_bool_t __connman_dbus_append_objpath_dict_array(DBusMessage *msg,
        return TRUE;
 }
 
-struct selinux_data {
-       connman_dbus_get_context_cb_t func;
+dbus_bool_t __connman_dbus_append_objpath_array(DBusMessage *msg,
+                       connman_dbus_append_cb_t function, void *user_data)
+{
+       DBusMessageIter iter, array;
+
+       if (!msg || !function)
+               return FALSE;
+
+       dbus_message_iter_init_append(msg, &iter);
+       dbus_message_iter_open_container(&iter, DBUS_TYPE_ARRAY,
+                               DBUS_TYPE_OBJECT_PATH_AS_STRING, &array);
+
+       function(&array, user_data);
+
+       dbus_message_iter_close_container(&iter, &array);
+
+       return TRUE;
+}
+
+struct callback_data {
+       void *cb;
        void *user_data;
 };
 
+static void get_connection_unix_user_reply(DBusPendingCall *call,
+                                               void *user_data)
+{
+       struct callback_data *data = user_data;
+       connman_dbus_get_connection_unix_user_cb_t cb = data->cb;
+       DBusMessageIter iter;
+       DBusMessage *reply;
+       int err = 0;
+       unsigned int uid = 0;
+
+       reply = dbus_pending_call_steal_reply(call);
+
+       if (dbus_message_get_type(reply) == DBUS_MESSAGE_TYPE_ERROR) {
+               DBG("Failed to retrieve UID");
+               err = -EIO;
+               goto done;
+       }
+
+       if (!dbus_message_has_signature(reply, "u")) {
+               DBG("Message signature is wrong");
+               err = -EINVAL;
+               goto done;
+       }
+
+       dbus_message_iter_init(reply, &iter);
+       dbus_message_iter_get_basic(&iter, &uid);
+
+done:
+       (*cb)(uid, data->user_data, err);
+
+       dbus_message_unref(reply);
+
+       dbus_pending_call_unref(call);
+}
+
+int connman_dbus_get_connection_unix_user(DBusConnection *connection,
+                               const char *bus_name,
+                               connman_dbus_get_connection_unix_user_cb_t func,
+                               void *user_data)
+{
+       struct callback_data *data;
+       DBusPendingCall *call;
+       DBusMessage *msg = NULL;
+       int err;
+
+       data = g_try_new0(struct callback_data, 1);
+       if (!data) {
+               DBG("Can't allocate data structure");
+               return -ENOMEM;
+       }
+
+       msg = dbus_message_new_method_call(DBUS_SERVICE_DBUS, DBUS_PATH_DBUS,
+                                       DBUS_INTERFACE_DBUS,
+                                       "GetConnectionUnixUser");
+       if (!msg) {
+               DBG("Can't allocate new message");
+               err = -ENOMEM;
+               goto err;
+       }
+
+       dbus_message_append_args(msg, DBUS_TYPE_STRING, &bus_name,
+                                       DBUS_TYPE_INVALID);
+
+       if (!dbus_connection_send_with_reply(connection, msg, &call, -1)) {
+               DBG("Failed to execute method call");
+               err = -EINVAL;
+               goto err;
+       }
+
+       if (!call) {
+               DBG("D-Bus connection not available");
+               err = -EINVAL;
+               goto err;
+       }
+
+       data->cb = func;
+       data->user_data = user_data;
+
+       dbus_pending_call_set_notify(call, get_connection_unix_user_reply,
+                                                       data, g_free);
+
+       dbus_message_unref(msg);
+
+       return 0;
+
+err:
+       dbus_message_unref(msg);
+       g_free(data);
+
+       return err;
+}
+
 static unsigned char *parse_context(DBusMessage *msg)
 {
        DBusMessageIter iter, array;
@@ -429,7 +530,7 @@ static unsigned char *parse_context(DBusMessage *msg)
                return NULL;
 
        ctx = g_try_malloc0(size + 1);
-       if (ctx == NULL)
+       if (!ctx)
                return NULL;
 
        p = ctx;
@@ -448,7 +549,8 @@ static unsigned char *parse_context(DBusMessage *msg)
 
 static void selinux_get_context_reply(DBusPendingCall *call, void *user_data)
 {
-       struct selinux_data *data = user_data;
+       struct callback_data *data = user_data;
+       connman_dbus_get_context_cb_t cb = data->cb;
        DBusMessage *reply;
        unsigned char *context = NULL;
        int err = 0;
@@ -461,7 +563,7 @@ static void selinux_get_context_reply(DBusPendingCall *call, void *user_data)
                goto done;
        }
 
-       if (dbus_message_has_signature(reply, "ay") == FALSE) {
+       if (!dbus_message_has_signature(reply, "ay")) {
                DBG("Message signature is wrong");
                err = -EINVAL;
                goto done;
@@ -470,7 +572,7 @@ static void selinux_get_context_reply(DBusPendingCall *call, void *user_data)
        context = parse_context(reply);
 
 done:
-       (*data->func)(context, data->user_data, err);
+       (*cb)(context, data->user_data, err);
 
        g_free(context);
 
@@ -484,16 +586,16 @@ int connman_dbus_get_selinux_context(DBusConnection *connection,
                                connman_dbus_get_context_cb_t func,
                                void *user_data)
 {
-       struct selinux_data *data;
+       struct callback_data *data;
        DBusPendingCall *call;
        DBusMessage *msg = NULL;
        int err;
 
-       if (func == NULL)
+       if (!func)
                return -EINVAL;
 
-       data = g_try_new0(struct selinux_data, 1);
-       if (data == NULL) {
+       data = g_try_new0(struct callback_data, 1);
+       if (!data) {
                DBG("Can't allocate data structure");
                return -ENOMEM;
        }
@@ -501,7 +603,7 @@ int connman_dbus_get_selinux_context(DBusConnection *connection,
        msg = dbus_message_new_method_call(DBUS_SERVICE_DBUS, DBUS_PATH_DBUS,
                                        DBUS_INTERFACE_DBUS,
                                        "GetConnectionSELinuxSecurityContext");
-       if (msg == NULL) {
+       if (!msg) {
                DBG("Can't allocate new message");
                err = -ENOMEM;
                goto err;
@@ -510,20 +612,19 @@ int connman_dbus_get_selinux_context(DBusConnection *connection,
        dbus_message_append_args(msg, DBUS_TYPE_STRING, &service,
                                        DBUS_TYPE_INVALID);
 
-       if (dbus_connection_send_with_reply(connection, msg,
-                                               &call, -1) == FALSE) {
+       if (!dbus_connection_send_with_reply(connection, msg, &call, -1)) {
                DBG("Failed to execute method call");
                err = -EINVAL;
                goto err;
        }
 
-       if (call == NULL) {
+       if (!call) {
                DBG("D-Bus connection not available");
                err = -EINVAL;
                goto err;
        }
 
-       data->func = func;
+       data->cb = func;
        data->user_data = user_data;
 
        dbus_pending_call_set_notify(call, selinux_get_context_reply,
@@ -540,9 +641,41 @@ err:
        return err;
 }
 
+void connman_dbus_reply_pending(DBusMessage *pending,
+                                       int error, const char *path)
+{
+       if (pending) {
+               if (error > 0) {
+                       DBusMessage *reply;
+
+                       reply = __connman_error_failed(pending, error);
+                       if (reply)
+                               g_dbus_send_message(connection, reply);
+               } else {
+                       const char *sender;
+
+                       sender = dbus_message_get_interface(pending);
+                       if (!path)
+                               path = dbus_message_get_path(pending);
+
+                       DBG("sender %s path %s", sender, path);
+
+                       if (g_strcmp0(sender, CONNMAN_MANAGER_INTERFACE) == 0)
+                               g_dbus_send_reply(connection, pending,
+                                       DBUS_TYPE_OBJECT_PATH, &path,
+                                                       DBUS_TYPE_INVALID);
+                       else
+                               g_dbus_send_reply(connection, pending,
+                                                       DBUS_TYPE_INVALID);
+               }
+
+               dbus_message_unref(pending);
+       }
+}
+
 DBusConnection *connman_dbus_get_connection(void)
 {
-       if (connection == NULL)
+       if (!connection)
                return NULL;
 
        return dbus_connection_ref(connection);
@@ -554,7 +687,7 @@ int __connman_dbus_init(DBusConnection *conn)
 
        connection = conn;
 
-       return 0;
+       return g_dbus_attach_object_manager(conn);
 }
 
 void __connman_dbus_cleanup(void)