diff --git a/coccinelle/empty-to-root.cocci b/coccinelle/empty-to-root.cocci new file mode 100644 index 00000000000..45627c98019 --- /dev/null +++ b/coccinelle/empty-to-root.cocci @@ -0,0 +1,11 @@ +@@ +expression s; +@@ +- if (empty_or_root(s)) +- s = "/"; ++ s = empty_to_root(s); +@@ +expression s; +@@ +- (empty_or_root(s) ? "/" : s) ++ empty_to_root(s) diff --git a/src/cryptsetup/cryptsetup-generator.c b/src/cryptsetup/cryptsetup-generator.c index 127b3c5380c..2197160c0fa 100644 --- a/src/cryptsetup/cryptsetup-generator.c +++ b/src/cryptsetup/cryptsetup-generator.c @@ -369,9 +369,6 @@ static crypto_device *get_crypto_device(const char *uuid) { if (!d) return NULL; - d->create = false; - d->keyfile = d->options = d->name = NULL; - d->uuid = strdup(uuid); if (!d->uuid) return mfree(d); diff --git a/src/shared/bus-unit-procs.c b/src/shared/bus-unit-procs.c index 054a1410acc..b21fe393265 100644 --- a/src/shared/bus-unit-procs.c +++ b/src/shared/bus-unit-procs.c @@ -31,8 +31,7 @@ static int add_cgroup(Hashmap *cgroups, const char *path, bool is_const, struct assert(cgroups); assert(ret); - if (empty_or_root(path)) - path = "/"; + path = empty_to_root(path); cg = hashmap_get(cgroups, path); if (cg) { @@ -48,8 +47,6 @@ static int add_cgroup(Hashmap *cgroups, const char *path, bool is_const, struct return -EINVAL; pp = strndupa(path, e - path); - if (!pp) - return -ENOMEM; r = add_cgroup(cgroups, pp, false, &parent); if (r < 0) @@ -150,8 +147,7 @@ static int dump_processes( assert(prefix); - if (empty_or_root(cgroup_path)) - cgroup_path = "/"; + cgroup_path = empty_to_root(cgroup_path); cg = hashmap_get(cgroups, cgroup_path); if (!cg)