X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Flibgdbus%2Fdbus-systemd.c;h=4597e158e33fad89dbbde236544f7c867796bf63;hb=7518a195e9d9f2f45a5f36b31350fed299b2fcab;hp=d77f733f26abe406690e34001479df8b31fdedb0;hpb=5733b3b5999a7c3f8d47ddea3d4b84c926d62434;p=platform%2Fcore%2Fsystem%2Flibsyscommon.git diff --git a/src/libgdbus/dbus-systemd.c b/src/libgdbus/dbus-systemd.c index d77f733..4597e15 100644 --- a/src/libgdbus/dbus-systemd.c +++ b/src/libgdbus/dbus-systemd.c @@ -25,9 +25,9 @@ #include "shared/log.h" -#define SYSTEMD_DBUS_SERVICE "org.freedesktop.systemd1" +#define SYSTEMD_DBUS_SERVICE "org.freedesktop.systemd1" #define SYSTEMD_DBUS_PATH "/org/freedesktop/systemd1" -#define SYSTEMD_DBUS_UNIT_PATH "/org/freedesktop/systemd1/unit/" +#define SYSTEMD_DBUS_UNIT_PATH "/org/freedesktop/systemd1/unit/" #define SYSTEMD_DBUS_MANAGER_IFACE "org.freedesktop.systemd1.Manager" #define SYSTEMD_DBUS_UNIT_IFACE "org.freedesktop.systemd1.Unit" @@ -36,17 +36,17 @@ #define DBUS_IFACE_DBUS_PROPERTIES "org.freedesktop.DBus.Properties" -#define SUFFIX_SERVICE ".service" -#define SUFFIX_SOCKET ".socket" -#define SUFFIX_BUSNAME ".busname" -#define SUFFIX_TARGET ".target" -#define SUFFIX_DEVICE ".device" -#define SUFFIX_MOUNT ".mount" -#define SUFFIX_SWAP ".swap" -#define SUFFIX_TIMER ".timer" -#define SUFFIX_PATH ".path" -#define SUFFIX_SLICE ".slice" -#define SUFFIX_SCOPE ".scope" +#define SUFFIX_SERVICE ".service" +#define SUFFIX_SOCKET ".socket" +#define SUFFIX_BUSNAME ".busname" +#define SUFFIX_TARGET ".target" +#define SUFFIX_DEVICE ".device" +#define SUFFIX_MOUNT ".mount" +#define SUFFIX_SWAP ".swap" +#define SUFFIX_TIMER ".timer" +#define SUFFIX_PATH ".path" +#define SUFFIX_SLICE ".slice" +#define SUFFIX_SCOPE ".scope" #define UNIT_NAME_MAX 256 @@ -56,12 +56,12 @@ typedef struct { } unitinfo; static void _cb_JobRemoved(GDBusConnection *conn, - const char *sender, - const char *path, - const char *iface, - const char *name, - GVariant *param, - gpointer data) + const char *sender, + const char *path, + const char *iface, + const char *name, + GVariant *param, + gpointer data) { sig_ctx *ctx = data; gchar *job_id = NULL; @@ -102,7 +102,9 @@ err: g_free(unit_name); } -static int _systemd_control_unit_wait(const char *method, const char *name, int timeout_msec) +static int _systemd_control_unit_wait(const char *method, + const char *name, + int timeout_msec) { GVariant *reply = NULL; gchar *objpath = NULL; @@ -118,17 +120,23 @@ static int _systemd_control_unit_wait(const char *method, const char *name, int _I("Starting: %s %s", method, name); /* synchronous signal subscription */ - ret = subscribe_dbus_signal_ctx(NULL, ctx, SYSTEMD_DBUS_SERVICE, SYSTEMD_DBUS_PATH, SYSTEMD_DBUS_IFACE_MANAGER, "JobRemoved", _cb_JobRemoved); + ret = subscribe_dbus_signal_ctx(NULL, + ctx, + SYSTEMD_DBUS_SERVICE, + SYSTEMD_DBUS_PATH, + SYSTEMD_DBUS_IFACE_MANAGER, + "JobRemoved", + _cb_JobRemoved); if (ret == 0) { ret = -1; goto finish; } reply = dbus_handle_method_sync_with_reply_var(SYSTEMD_DBUS_DEST, - SYSTEMD_DBUS_PATH, - SYSTEMD_DBUS_MANAGER_IFACE, - method, - g_variant_new("(ss)", name, "replace")); + SYSTEMD_DBUS_PATH, + SYSTEMD_DBUS_MANAGER_IFACE, + method, + g_variant_new("(ss)", name, "replace")); if (!reply || !dh_get_param_from_var(reply, "(o)", &objpath)) { _E("fail (%s): no message", method); ret = -EBADMSG; @@ -175,10 +183,10 @@ static int _systemd_control_unit_async(const char *method, const char *name) _I("Starting: %s %s", method, name); reply = dbus_handle_method_sync_with_reply_var(SYSTEMD_DBUS_DEST, - SYSTEMD_DBUS_PATH, - SYSTEMD_DBUS_MANAGER_IFACE, - method, - g_variant_new("(ss)", name, "replace")); + SYSTEMD_DBUS_PATH, + SYSTEMD_DBUS_MANAGER_IFACE, + method, + g_variant_new("(ss)", name, "replace")); if (!reply || !dh_get_param_from_var(reply, "(o)", &objpath)) { _E("fail (%s): no message", method); @@ -195,7 +203,9 @@ finish: return ret; } -static int _change_suffix(const char *name, const char *suffix, char **new_name) +static int _change_suffix(const char *name, + const char *suffix, + char **new_name) { char *buf = NULL; char *ext = NULL; @@ -203,7 +213,8 @@ static int _change_suffix(const char *name, const char *suffix, char **new_name) int ret = 0; if (!name || !suffix || !new_name) { - _E("Wrong param name:%s, suffix:%s, new_name:%s", name, suffix, new_name); + _E("Wrong param name:%s, suffix:%s, new_name:%s (null)", + name ? : "(null)", suffix ? : "(null)", new_name ? "not" : ""); return -EINVAL; } @@ -275,17 +286,20 @@ Start or Stop systemd unit. @param suffix: (nullable): change extension of unit name, or %NULL @param timeout_msec: the timeout in milliseconds, -1 to use the default @param method: method name, "StartUnit" or "StopUnit" -@param sync: %TRUE +@param wait: %TRUE Returns: the exit status */ -static int _systemd_control_unit(const char *name, const char *suffix, - int timeout_msec, const char *method, int wait) +static int _systemd_control_unit(const char *name, + const char *suffix, + int timeout_msec, + const char *method, + int wait) { char *new_name = NULL; int ret = 0; if (!name || !method) { - _E("Wrong param name %s, method %s", name, method); + _E("Wrong param name %s, method %s", name ? : "(null)", method ? : "(null)"); return -EINVAL; } if (timeout_msec < -1) { @@ -395,10 +409,10 @@ GVariant *systemd_get_manager_property(const char *property) return NULL; reply = dbus_handle_method_sync_with_reply_var(SYSTEMD_DBUS_DEST, - SYSTEMD_DBUS_PATH, - DBUS_IFACE_DBUS_PROPERTIES, - "Get", - g_variant_new("(ss)", SYSTEMD_DBUS_MANAGER_IFACE, property)); + SYSTEMD_DBUS_PATH, + DBUS_IFACE_DBUS_PROPERTIES, + "Get", + g_variant_new("(ss)", SYSTEMD_DBUS_MANAGER_IFACE, property)); if (!reply || !dh_get_param_from_var(reply, "(v)", &val)) _E("Failed to get variant"); if (reply) @@ -407,8 +421,7 @@ GVariant *systemd_get_manager_property(const char *property) return val; } -GVariant *systemd_get_unit_property(const char *unit, - const char *property) +GVariant *systemd_get_unit_property(const char *unit, const char *property) { char *escaped; GVariant *reply = NULL; @@ -420,10 +433,10 @@ GVariant *systemd_get_unit_property(const char *unit, escaped = systemd_get_unit_dbus_path(unit); reply = dbus_handle_method_sync_with_reply_var(SYSTEMD_DBUS_DEST, - escaped, - DBUS_IFACE_DBUS_PROPERTIES, - "Get", - g_variant_new("(ss)", SYSTEMD_DBUS_UNIT_IFACE, property)); + escaped, + DBUS_IFACE_DBUS_PROPERTIES, + "Get", + g_variant_new("(ss)", SYSTEMD_DBUS_UNIT_IFACE, property)); if (!reply || !dh_get_param_from_var(reply, "(v)", &val)) _E("Failed to get variant"); @@ -434,8 +447,7 @@ GVariant *systemd_get_unit_property(const char *unit, return val; } -GVariant *systemd_get_service_property(const char *unit, - const char *property) +GVariant *systemd_get_service_property(const char *unit, const char *property) { char *escaped; GVariant *reply = NULL; @@ -447,10 +459,10 @@ GVariant *systemd_get_service_property(const char *unit, escaped = systemd_get_unit_dbus_path(unit); reply = dbus_handle_method_sync_with_reply_var(SYSTEMD_DBUS_DEST, - escaped, - DBUS_IFACE_DBUS_PROPERTIES, - "Get", - g_variant_new("(ss)", SYSTEMD_DBUS_SERVICE_IFACE, property)); + escaped, + DBUS_IFACE_DBUS_PROPERTIES, + "Get", + g_variant_new("(ss)", SYSTEMD_DBUS_SERVICE_IFACE, property)); if (!reply || !dh_get_param_from_var(reply, "(v)", &val)) _E("Failed to get variant"); if (reply)