]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
pid1: replace asprintf() with strjoin() 13256/head
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Sat, 3 Aug 2019 15:46:56 +0000 (17:46 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Sat, 3 Aug 2019 15:46:56 +0000 (17:46 +0200)
It's nicer. And coverity doesn't need to complain about unchecked return
value (CID#1401780).

src/core/dbus-execute.c

index c816569f2b6fb6604cea677318a6d6ebf4d7174a..85c310cd33d0585fb5f77fa16bac779913632bdd 100644 (file)
@@ -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(