summaryrefslogtreecommitdiff
path: root/src/task.c
diff options
context:
space:
mode:
authorZhang zhengguang <zhengguang.zhang@intel.com>2014-07-17 10:37:39 +0800
committerZhang zhengguang <zhengguang.zhang@intel.com>2014-07-17 10:37:39 +0800
commit1b9d0a62f59bb48c8deb2f0b98d9acdffdd9abe7 (patch)
tree6e991827d28537f7f40f20786c2354fd04a9fdad /src/task.c
parentfbe905ab58ecc31fe64c410c5f580cadc30e7f04 (diff)
downloadconnman-1b9d0a62f59bb48c8deb2f0b98d9acdffdd9abe7.tar.gz
connman-1b9d0a62f59bb48c8deb2f0b98d9acdffdd9abe7.tar.bz2
connman-1b9d0a62f59bb48c8deb2f0b98d9acdffdd9abe7.zip
Imported Upstream version 1.24upstream/1.24
Diffstat (limited to 'src/task.c')
-rw-r--r--src/task.c43
1 files changed, 21 insertions, 22 deletions
diff --git a/src/task.c b/src/task.c
index 60e336ac..8b9e1d93 100644
--- a/src/task.c
+++ b/src/task.c
@@ -102,7 +102,7 @@ struct connman_task *connman_task_create(const char *program)
DBG("");
task = g_try_new0(struct connman_task, 1);
- if (task == NULL)
+ if (!task)
return NULL;
counter = __sync_fetch_and_add(&task_counter, 1);
@@ -167,7 +167,7 @@ int connman_task_add_argument(struct connman_task *task,
DBG("task %p arg %s", task, name);
- if (name == NULL)
+ if (!name)
return -EINVAL;
str = g_strdup(name);
@@ -175,7 +175,7 @@ int connman_task_add_argument(struct connman_task *task,
va_start(ap, format);
- if (format != NULL) {
+ if (format) {
str = g_strdup_vprintf(format, ap);
g_ptr_array_add(task->argv, str);
}
@@ -202,7 +202,7 @@ int connman_task_add_variable(struct connman_task *task,
DBG("task %p key %s", task, key);
- if (key == NULL)
+ if (!key)
return -EINVAL;
va_start(ap, format);
@@ -234,7 +234,7 @@ int connman_task_set_notify(struct connman_task *task, const char *member,
DBG("task %p", task);
notify = g_try_new0(struct notify_data, 1);
- if (notify == NULL)
+ if (!notify)
return -ENOMEM;
notify->func = function;
@@ -293,7 +293,7 @@ int connman_task_run(struct connman_task *task,
int *stdin_fd, int *stdout_fd, int *stderr_fd)
{
GSpawnFlags flags = G_SPAWN_DO_NOT_REAP_CHILD;
- gboolean result;
+ bool result;
char **argv, **envp;
DBG("task %p", task);
@@ -301,20 +301,20 @@ int connman_task_run(struct connman_task *task,
if (task->pid > 0)
return -EALREADY;
- if (stdout_fd == NULL)
+ if (!stdout_fd)
flags |= G_SPAWN_STDOUT_TO_DEV_NULL;
- if (stderr_fd == NULL)
+ if (!stderr_fd)
flags |= G_SPAWN_STDERR_TO_DEV_NULL;
task->exit_func = function;
task->exit_data = user_data;
- if (g_ptr_array_index(task->argv, task->argv->len - 1) != NULL)
+ if (g_ptr_array_index(task->argv, task->argv->len - 1))
g_ptr_array_add(task->argv, NULL);
- if (task->envp->len == 0 || g_ptr_array_index(task->envp,
- task->envp->len - 1) != NULL) {
+ if (task->envp->len == 0 ||
+ g_ptr_array_index(task->envp, task->envp->len - 1)) {
if (g_hash_table_size(task->notify) > 0) {
const char *busname;
char *str;
@@ -340,7 +340,7 @@ int connman_task_run(struct connman_task *task,
result = g_spawn_async_with_pipes(NULL, argv, envp, flags,
task_setup, task, &task->pid,
stdin_fd, stdout_fd, stderr_fd, NULL);
- if (result == FALSE) {
+ if (!result) {
connman_error("Failed to spawn %s", argv[0]);
return -EIO;
}
@@ -419,39 +419,38 @@ static DBusHandlerResult task_filter(DBusConnection *conn,
if (dbus_message_get_type(message) != DBUS_MESSAGE_TYPE_METHOD_CALL)
return DBUS_HANDLER_RESULT_NOT_YET_HANDLED;
- if (dbus_message_has_interface(message,
- CONNMAN_TASK_INTERFACE) == FALSE)
+ if (!dbus_message_has_interface(message, CONNMAN_TASK_INTERFACE))
return DBUS_HANDLER_RESULT_NOT_YET_HANDLED;
path = dbus_message_get_path(message);
- if (path == NULL)
+ if (!path)
return DBUS_HANDLER_RESULT_NOT_YET_HANDLED;
task = g_hash_table_lookup(task_hash, path);
- if (task == NULL)
+ if (!task)
return DBUS_HANDLER_RESULT_NOT_YET_HANDLED;
member = dbus_message_get_member(message);
- if (member == NULL)
+ if (!member)
goto send_reply;
notify = g_hash_table_lookup(task->notify, member);
- if (notify == NULL)
+ if (!notify)
goto send_reply;
if (notify->func)
reply = notify->func(task, message, notify->data);
send_reply:
- if (dbus_message_get_no_reply(message) == FALSE &&
- reply == NULL) {
+ if (!dbus_message_get_no_reply(message) &&
+ !reply) {
reply = dbus_message_new_method_return(message);
- if (reply == NULL)
+ if (!reply)
return DBUS_HANDLER_RESULT_NEED_MEMORY;
}
- if (reply != NULL) {
+ if (reply) {
dbus_connection_send(conn, reply, NULL);
dbus_message_unref(reply);