From ec04aef44225ab0b833b427b43ce8ab69385716d Mon Sep 17 00:00:00 2001 From: Topi Miettinen Date: Thu, 2 Jan 2020 19:59:48 +0200 Subject: [PATCH] dbus-execute: avoid extra strdup() bind_mount_add does the strdup(), so we can avoid strdup()ing the strings. --- src/core/dbus-execute.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/core/dbus-execute.c b/src/core/dbus-execute.c index 1d0bc1ede3c..c6772ba8431 100644 --- a/src/core/dbus-execute.c +++ b/src/core/dbus-execute.c @@ -2379,7 +2379,7 @@ int bus_exec_context_set_transient_property( return 1; } else if (STR_IN_SET(name, "BindPaths", "BindReadOnlyPaths")) { - const char *source, *destination; + char *source, *destination; int ignore_enoent; uint64_t mount_flags; bool empty = true; @@ -2400,8 +2400,8 @@ int bus_exec_context_set_transient_property( if (!UNIT_WRITE_FLAGS_NOOP(flags)) { r = bind_mount_add(&c->bind_mounts, &c->n_bind_mounts, &(BindMount) { - .source = strdup(source), - .destination = strdup(destination), + .source = source, + .destination = destination, .read_only = !!strstr(name, "ReadOnly"), .recursive = !!(mount_flags & MS_REC), .ignore_enoent = ignore_enoent, -- 2.47.3