pid1: replace asprintf() with strjoin()

It's nicer. And coverity doesn't need to complain about unchecked return
value (CID#1401780).

diff --git a/src/core/dbus-execute.c b/src/core/dbus-execute.c
index c816569f2b6fb6604cea677318a6d6ebf4d7174a..85c310cd33d0585fb5f77fa16bac779913632bdd 100644 (file)
--- a/src/core/dbus-execute.c
+++ b/src/core/dbus-execute.c
@@ -1000,16 +1000,11 @@ int bus_property_get_exec_ex_command_list(
 }
 
 static char *exec_command_flags_to_exec_chars(ExecCommandFlags flags) {
-        char *res = NULL;
-
-        asprintf(&res, "%s%s%s%s%s",
-                FLAGS_SET(flags, EXEC_COMMAND_IGNORE_FAILURE)   ? "-" : "",
-                FLAGS_SET(flags, EXEC_COMMAND_NO_ENV_EXPAND)    ? ":" : "",
-                FLAGS_SET(flags, EXEC_COMMAND_FULLY_PRIVILEGED) ? "+" : "",
-                FLAGS_SET(flags, EXEC_COMMAND_NO_SETUID)        ? "!" : "",
-                FLAGS_SET(flags, EXEC_COMMAND_AMBIENT_MAGIC)    ? "!!" : "");
-
-        return res;
+        return strjoin(FLAGS_SET(flags, EXEC_COMMAND_IGNORE_FAILURE)   ? "-" : "",
+                       FLAGS_SET(flags, EXEC_COMMAND_NO_ENV_EXPAND)    ? ":" : "",
+                       FLAGS_SET(flags, EXEC_COMMAND_FULLY_PRIVILEGED) ? "+" : "",
+                       FLAGS_SET(flags, EXEC_COMMAND_NO_SETUID)        ? "!" : "",
+                       FLAGS_SET(flags, EXEC_COMMAND_AMBIENT_MAGIC)    ? "!!" : "");
 }
 
 int bus_set_transient_exec_command(