tree-wide: use strv_extend_strv_consume() where appropriate

diff --git a/src/basic/env-util.c b/src/basic/env-util.c
index cf2c47d2142728bc61fae4f5e30ec51429a5ece9..4c06d1a1650f59f081dc0e35a0a5f3c7d7a5d39a 100644 (file)
--- a/src/basic/env-util.c
+++ b/src/basic/env-util.c
@@ -796,10 +796,10 @@ int replace_env_full(
                                         t = v;
                                 }
 
-                                r = strv_extend_strv(&unset_variables, u, /* filter_duplicates= */ true);
+                                r = strv_extend_strv_consume(&unset_variables, TAKE_PTR(u), /* filter_duplicates= */ true);
                                 if (r < 0)
                                         return r;
-                                r = strv_extend_strv(&bad_variables, b, /* filter_duplicates= */ true);
+                                r = strv_extend_strv_consume(&bad_variables, TAKE_PTR(b), /* filter_duplicates= */ true);
                                 if (r < 0)
                                         return r;
 
@@ -931,11 +931,11 @@ int replace_env_argv(
                         return r;
                 n[++k] = NULL;
 
-                r = strv_extend_strv(&unset_variables, u, /* filter_duplicates= */ true);
+                r = strv_extend_strv_consume(&unset_variables, TAKE_PTR(u), /* filter_duplicates= */ true);
                 if (r < 0)
                         return r;
 
-                r = strv_extend_strv(&bad_variables, b, /*filter_duplicates= */ true);
+                r = strv_extend_strv_consume(&bad_variables, TAKE_PTR(b), /* filter_duplicates= */ true);
                 if (r < 0)
                         return r;
         }

diff --git a/src/basic/strv.c b/src/basic/strv.c
index 542883ea3ff51ebd9b3d837dd2f68e62688bc548..f9628835979e437d2437fceb82d707aea1f86821 100644 (file)
--- a/src/basic/strv.c
+++ b/src/basic/strv.c
@@ -405,7 +405,7 @@ int strv_split_full(char ***t, const char *s, const char *separators, ExtractFla
 }
 
 int strv_split_and_extend_full(char ***t, const char *s, const char *separators, bool filter_duplicates, ExtractFlags flags) {
-        _cleanup_strv_free_ char **l = NULL;
+        char **l;
         int r;
 
         assert(t);
@@ -415,7 +415,7 @@ int strv_split_and_extend_full(char ***t, const char *s, const char *separators,
         if (r < 0)
                 return r;
 
-        r = strv_extend_strv(t, l, filter_duplicates);
+        r = strv_extend_strv_consume(t, l, filter_duplicates);
         if (r < 0)
                 return r;
 

diff --git a/src/core/execute.c b/src/core/execute.c
index 5e0e0b3a62a581306b3de0c53cff052992b2a446..30fcffcc5bdf03bde5b95463516a7d597e7e198a 100644 (file)
--- a/src/core/execute.c
+++ b/src/core/execute.c
@@ -2046,7 +2046,7 @@ int exec_command_set(ExecCommand *c, const char *path, ...) {
 }
 
 int exec_command_append(ExecCommand *c, const char *path, ...) {
-        _cleanup_strv_free_ char **l = NULL;
+        char **l;
         va_list ap;
         int r;
 
@@ -2060,7 +2060,7 @@ int exec_command_append(ExecCommand *c, const char *path, ...) {
         if (!l)
                 return -ENOMEM;
 
-        r = strv_extend_strv(&c->argv, l, false);
+        r = strv_extend_strv_consume(&c->argv, l, /* filter_duplicates = */ false);
         if (r < 0)
                 return r;
 

diff --git a/src/core/load-dropin.c b/src/core/load-dropin.c
index dc9c44e6d64835d13d8d1a9ed9d632270cf796e2..5c1ccc966f66a43a08b5c233c90e9930c8f3da7f 100644 (file)
--- a/src/core/load-dropin.c
+++ b/src/core/load-dropin.c
@@ -106,13 +106,9 @@ int unit_load_dropin(Unit *u) {
         if (r <= 0)
                 return 0;
 
-        if (!u->dropin_paths)
-                u->dropin_paths = TAKE_PTR(l);
-        else {
-                r = strv_extend_strv(&u->dropin_paths, l, true);
-                if (r < 0)
-                        return log_oom();
-        }
+        r = strv_extend_strv_consume(&u->dropin_paths, TAKE_PTR(l), /* filter_duplicates = */ true);
+        if (r < 0)
+                return log_oom();
 
         u->dropin_mtime = 0;
         STRV_FOREACH(f, u->dropin_paths) {

diff --git a/src/core/load-fragment.c b/src/core/load-fragment.c
index d8fead1ca50d790b3a3995e3c217509970f226d2..e49276eae5c705cc31f97439a2de5fa352e6c60d 100644 (file)
--- a/src/core/load-fragment.c
+++ b/src/core/load-fragment.c
@@ -2779,10 +2779,8 @@ int config_parse_pass_environ(
                 void *data,
                 void *userdata) {
 
-        _cleanup_strv_free_ char **n = NULL;
+        char ***passenv = ASSERT_PTR(data);
         const Unit *u = userdata;
-        char*** passenv = ASSERT_PTR(data);
-        size_t nlen = 0;
         int r;
 
         assert(filename);
@@ -2795,6 +2793,8 @@ int config_parse_pass_environ(
                 return 0;
         }
 
+        _cleanup_strv_free_ char **n = NULL;
+
         for (const char *p = rvalue;;) {
                 _cleanup_free_ char *word = NULL, *k = NULL;
 
@@ -2825,18 +2825,13 @@ int config_parse_pass_environ(
                         continue;
                 }
 
-                if (!GREEDY_REALLOC(n, nlen + 2))
+                if (strv_consume(&n, TAKE_PTR(k)) < 0)
                         return log_oom();
-
-                n[nlen++] = TAKE_PTR(k);
-                n[nlen] = NULL;
         }
 
-        if (n) {
-                r = strv_extend_strv(passenv, n, true);
-                if (r < 0)
-                        return log_oom();
-        }
+        r = strv_extend_strv_consume(passenv, TAKE_PTR(n), /* filter_duplicates = */ true);
+        if (r < 0)
+                return log_oom();
 
         return 0;
 }
@@ -2853,10 +2848,8 @@ int config_parse_unset_environ(
                 void *data,
                 void *userdata) {
 
-        _cleanup_strv_free_ char **n = NULL;
-        char*** unsetenv = ASSERT_PTR(data);
+        char ***unsetenv = ASSERT_PTR(data);
         const Unit *u = userdata;
-        size_t nlen = 0;
         int r;
 
         assert(filename);
@@ -2869,6 +2862,8 @@ int config_parse_unset_environ(
                 return 0;
         }
 
+        _cleanup_strv_free_ char **n = NULL;
+
         for (const char *p = rvalue;;) {
                 _cleanup_free_ char *word = NULL, *k = NULL;
 
@@ -2899,18 +2894,13 @@ int config_parse_unset_environ(
                         continue;
                 }
 
-                if (!GREEDY_REALLOC(n, nlen + 2))
+                if (strv_consume(&n, TAKE_PTR(k)) < 0)
                         return log_oom();
-
-                n[nlen++] = TAKE_PTR(k);
-                n[nlen] = NULL;
         }
 
-        if (n) {
-                r = strv_extend_strv(unsetenv, n, true);
-                if (r < 0)
-                        return log_oom();
-        }
+        r = strv_extend_strv_consume(unsetenv, TAKE_PTR(n), /* filter_duplicates = */ true);
+        if (r < 0)
+                return log_oom();
 
         return 0;
 }

diff --git a/src/home/homectl.c b/src/home/homectl.c
index a52bc6d4648a8abcdb037b6898fd9c06fbc4c664..c193715d96ea9920eacdb5c88475ff27b9cbdf69 100644 (file)
--- a/src/home/homectl.c
+++ b/src/home/homectl.c
@@ -3652,7 +3652,7 @@ static int parse_argv(int argc, char *argv[]) {
                                         return log_error_errno(r, "Failed to parse SSH authorized keys list: %m");
                         }
 
-                        r = strv_extend_strv(&l, add, true);
+                        r = strv_extend_strv_consume(&l, TAKE_PTR(add), /* filter_duplicates = */ true);
                         if (r < 0)
                                 return log_oom();
 

diff --git a/src/libsystemd-network/sd-dhcp6-lease.c b/src/libsystemd-network/sd-dhcp6-lease.c
index e5d65475881628794ae2d69f9059abfb9add8923..c2eb0db87def52b3d0956c68ae5452552a561e7a 100644 (file)
--- a/src/libsystemd-network/sd-dhcp6-lease.c
+++ b/src/libsystemd-network/sd-dhcp6-lease.c
@@ -424,7 +424,7 @@ int dhcp6_lease_add_domains(sd_dhcp6_lease *lease, const uint8_t *optval, size_t
         if (r < 0)
                 return r;
 
-        return strv_extend_strv(&lease->domains, domains, true);
+        return strv_extend_strv_consume(&lease->domains, TAKE_PTR(domains), /* filter_duplicates = */ true);
 }
 
 int sd_dhcp6_lease_get_domains(sd_dhcp6_lease *lease, char ***ret) {

diff --git a/src/libsystemd/sd-varlink/sd-varlink.c b/src/libsystemd/sd-varlink/sd-varlink.c
index a0aa12a38696d988c605010b5edf7cff2b8f9ebc..007746f64f61b1e3268a71af49340f3cc9c99028 100644 (file)
--- a/src/libsystemd/sd-varlink/sd-varlink.c
+++ b/src/libsystemd/sd-varlink/sd-varlink.c
@@ -410,7 +410,7 @@ static int varlink_connect_ssh_exec(sd_varlink **ret, const char *where) {
         full_cmdline = strv_new("ssh", "-e", "none", "-T", h, "env", "SYSTEMD_VARLINK_LISTEN=-");
         if (!full_cmdline)
                 return log_oom_debug();
-        r = strv_extend_strv(&full_cmdline, cmdline, /* filter_duplicates= */ false);
+        r = strv_extend_strv_consume(&full_cmdline, TAKE_PTR(cmdline), /* filter_duplicates= */ false);
         if (r < 0)
                 return log_oom_debug();
 

diff --git a/src/modules-load/modules-load.c b/src/modules-load/modules-load.c
index c1f818320ffb7048ec0c68dca3f53ceb49d65641..59daeb5cb014662b65ed85ccf8a8075a7a0a4589 100644 (file)
--- a/src/modules-load/modules-load.c
+++ b/src/modules-load/modules-load.c
@@ -23,19 +23,6 @@ static const char conf_file_dirs[] = CONF_PATHS_NULSTR("modules-load.d");
 
 STATIC_DESTRUCTOR_REGISTER(arg_proc_cmdline_modules, strv_freep);
 
-static int add_modules(const char *p) {
-        _cleanup_strv_free_ char **k = NULL;
-
-        k = strv_split(p, ",");
-        if (!k)
-                return log_oom();
-
-        if (strv_extend_strv(&arg_proc_cmdline_modules, k, true) < 0)
-                return log_oom();
-
-        return 0;
-}
-
 static int parse_proc_cmdline_item(const char *key, const char *value, void *data) {
         int r;
 
@@ -44,9 +31,9 @@ static int parse_proc_cmdline_item(const char *key, const char *value, void *dat
                 if (proc_cmdline_value_missing(key, value))
                         return 0;
 
-                r = add_modules(value);
+                r = strv_split_and_extend(&arg_proc_cmdline_modules, value, ",", /* filter_duplicates = */ true);
                 if (r < 0)
-                        return r;
+                        return log_error_errno(r, "Failed to parse modules_load= kernel command line option: %m");
         }
 
         return 0;

diff --git a/src/partition/repart.c b/src/partition/repart.c
index 660eef563e6ca3042cd08784d0eb617647f8c5ec..1c7ac5094e2fa8f1ebdf6251252096ffab4212f4 100644 (file)
--- a/src/partition/repart.c
+++ b/src/partition/repart.c
@@ -5361,13 +5361,13 @@ static int make_subvolumes_strv(const Partition *p, char ***ret) {
                         return log_oom();
 
         if (p->suppressing) {
-                _cleanup_strv_free_ char **suppressing = NULL;
+                char **suppressing;
 
                 r = make_subvolumes_strv(p->suppressing, &suppressing);
                 if (r < 0)
                         return r;
 
-                r = strv_extend_strv(&subvolumes, suppressing, /* filter_duplicates= */ true);
+                r = strv_extend_strv_consume(&subvolumes, suppressing, /* filter_duplicates= */ true);
                 if (r < 0)
                         return log_oom();
         }

diff --git a/src/portable/portable.c b/src/portable/portable.c
index af7d9552da45808ef094c3250141da955f094f54..f0c508c306f3a9c751cc35eefbfcc9031f28c990 100644 (file)
--- a/src/portable/portable.c
+++ b/src/portable/portable.c
@@ -721,13 +721,7 @@ static int extract_image_and_extensions(
 
                 e = strv_env_pairs_get(extension_release, "PORTABLE_PREFIXES");
                 if (e) {
-                        _cleanup_strv_free_ char **l = NULL;
-
-                        l = strv_split(e, WHITESPACE);
-                        if (!l)
-                                return -ENOMEM;
-
-                        r = strv_extend_strv(&valid_prefixes, l, true);
+                        r = strv_split_and_extend(&valid_prefixes, e, WHITESPACE, /* filter_duplicates = */ true);
                         if (r < 0)
                                 return r;
                 }

diff --git a/src/shared/bootspec.c b/src/shared/bootspec.c
index 50a07400fe3348e10d9d4f218840f150d4f2397f..87aa11ccdfc7880f4062e80fd3de1108381195db 100644 (file)
--- a/src/shared/bootspec.c
+++ b/src/shared/bootspec.c
@@ -176,7 +176,7 @@ static int parse_path_many(
                         return r;
         }
 
-        return strv_extend_strv(s, f, /* filter_duplicates= */ false);
+        return strv_extend_strv_consume(s, TAKE_PTR(f), /* filter_duplicates= */ false);
 }
 
 static int parse_tries(const char *fname, const char **p, unsigned *ret) {

diff --git a/src/shared/conf-parser.c b/src/shared/conf-parser.c
index 39e79ef3f6898085c82d66c790f6abed48daa67c..8b32f2a9bb2480c17f5983419c4b1d66bbebf73f 100644 (file)
--- a/src/shared/conf-parser.c
+++ b/src/shared/conf-parser.c
@@ -1343,7 +1343,7 @@ int config_parse_strv(
                         return log_oom();
         }
 
-        r = strv_extend_strv(sv, strv, /* filter_duplicates = */ ltype);
+        r = strv_extend_strv_consume(sv, TAKE_PTR(strv), /* filter_duplicates = */ ltype);
         if (r < 0)
                 return log_oom();
 

diff --git a/src/shared/module-util.c b/src/shared/module-util.c
index 9608b0a6713776fc6412dd2dd9c68461b20c9ce4..56e93e87323b0395038cf233c665fa7d50a251ee 100644 (file)
--- a/src/shared/module-util.c
+++ b/src/shared/module-util.c
@@ -49,25 +49,8 @@ int dlopen_libkmod(void) {
                         DLSYM_ARG(kmod_validate_resources));
 }
 
-static int denylist_modules(const char *p, char ***denylist) {
-        _cleanup_strv_free_ char **k = NULL;
-        int r;
-
-        assert(p);
-        assert(denylist);
-
-        k = strv_split(p, ",");
-        if (!k)
-                return -ENOMEM;
-
-        r = strv_extend_strv(denylist, k, /* filter_duplicates= */ true);
-        if (r < 0)
-                return r;
-
-        return 0;
-}
-
 static int parse_proc_cmdline_item(const char *key, const char *value, void *data) {
+        char ***denylist = ASSERT_PTR(data);
         int r;
 
         if (proc_cmdline_key_streq(key, "module_blacklist")) {
@@ -75,7 +58,7 @@ static int parse_proc_cmdline_item(const char *key, const char *value, void *dat
                 if (proc_cmdline_value_missing(key, value))
                         return 0;
 
-                r = denylist_modules(value, data);
+                r = strv_split_and_extend(denylist, value, ",", /* filter_duplicates = */ true);
                 if (r < 0)
                         return r;
         }

diff --git a/src/shared/user-record.c b/src/shared/user-record.c
index d6752f1c7bf3486b7bf9db7307f152863527d6fe..0de768cc59f3d164ed1a02b882796ead5c7b6d4e 100644 (file)
--- a/src/shared/user-record.c
+++ b/src/shared/user-record.c
@@ -626,16 +626,15 @@ static int json_dispatch_weight(const char *name, sd_json_variant *variant, sd_j
 }
 
 int json_dispatch_user_group_list(const char *name, sd_json_variant *variant, sd_json_dispatch_flags_t flags, void *userdata) {
+        char ***list = ASSERT_PTR(userdata);
         _cleanup_strv_free_ char **l = NULL;
-        char ***list = userdata;
-        sd_json_variant *e;
         int r;
 
         if (!sd_json_variant_is_array(variant))
                 return json_log(variant, flags, SYNTHETIC_ERRNO(EINVAL), "JSON field '%s' is not an array of strings.", strna(name));
 
+        sd_json_variant *e;
         JSON_VARIANT_ARRAY_FOREACH(e, variant) {
-
                 if (!sd_json_variant_is_string(e))
                         return json_log(e, flags, SYNTHETIC_ERRNO(EINVAL), "JSON array element is not a string.");
 
@@ -647,7 +646,7 @@ int json_dispatch_user_group_list(const char *name, sd_json_variant *variant, sd
                         return json_log(e, flags, r, "Failed to append array element: %m");
         }
 
-        r = strv_extend_strv(list, l, true);
+        r = strv_extend_strv_consume(list, TAKE_PTR(l), /* filter_duplicates = */ true);
         if (r < 0)
                 return json_log(variant, flags, r, "Failed to merge user/group arrays: %m");
 

diff --git a/src/systemctl/systemctl-util.c b/src/systemctl/systemctl-util.c
index 848012de6635c4930e2397f67a2172eae4de8deb..0b5881ccbbf629dbc875d36514173ce7a035dde4 100644 (file)
--- a/src/systemctl/systemctl-util.c
+++ b/src/systemctl/systemctl-util.c
@@ -739,14 +739,14 @@ int append_unit_dependencies(sd_bus *bus, char **names, char ***ret) {
         assert(ret);
 
         STRV_FOREACH(name, names) {
-                _cleanup_strv_free_ char **deps = NULL;
+                char **deps;
 
                 if (strv_extend(&with_deps, *name) < 0)
                         return log_oom();
 
                 (void) unit_get_dependencies(bus, *name, &deps);
 
-                if (strv_extend_strv(&with_deps, deps, true) < 0)
+                if (strv_extend_strv_consume(&with_deps, deps, /* filter_duplicates = */ true) < 0)
                         return log_oom();
         }
 

diff --git a/src/sysupdate/sysupdated.c b/src/sysupdate/sysupdated.c
index 546f52a96ce5304f54a76597125f959902447c02..f4ff472df13c453398ec587b2c0d2057a152dbab 100644 (file)
--- a/src/sysupdate/sysupdated.c
+++ b/src/sysupdate/sysupdated.c
@@ -1771,12 +1771,13 @@ static int method_list_appstream(sd_bus_message *msg, void *userdata, sd_bus_err
                 return r;
 
         HASHMAP_FOREACH(t, m->targets) {
-                _cleanup_strv_free_ char **target_appstream = NULL;
+                char **target_appstream;
+
                 r = target_get_appstream(t, &target_appstream);
                 if (r < 0)
                         return r;
 
-                r = strv_extend_strv(&urls, target_appstream, true);
+                r = strv_extend_strv_consume(&urls, target_appstream, /* filter_duplicates = */ true);
                 if (r < 0)
                         return r;
         }

diff --git a/src/udev/udevadm-settle.c b/src/udev/udevadm-settle.c
index f2b6c58b94001031a4ed50b42acb90c97d2df575..d47a9c3a07f2367758ba0964cbce6df2a4d9a272 100644 (file)
--- a/src/udev/udevadm-settle.c
+++ b/src/udev/udevadm-settle.c
@@ -128,7 +128,7 @@ static int emit_deprecation_warning(void) {
                                 NULL,
                                 &b);
 
-                r = strv_extend_strv(&a, b, true);
+                r = strv_extend_strv_consume(&a, TAKE_PTR(b), /* filter_duplicates = */ true);
                 if (r < 0)
                         return r;
         }

diff --git a/src/userdb/userdbctl.c b/src/userdb/userdbctl.c
index 1346e2de0126ec89a0451da8df7cf994a2a1a95d..5997f9604f0f400daf3a454ae0a2ff79a21d5b6e 100644 (file)
--- a/src/userdb/userdbctl.c
+++ b/src/userdb/userdbctl.c
@@ -1220,15 +1220,9 @@ static int parse_argv(int argc, char *argv[]) {
                         if (isempty(optarg))
                                 arg_services = strv_free(arg_services);
                         else {
-                                _cleanup_strv_free_ char **l = NULL;
-
-                                l = strv_split(optarg, ":");
-                                if (!l)
-                                        return log_oom();
-
-                                r = strv_extend_strv(&arg_services, l, true);
+                                r = strv_split_and_extend(&arg_services, optarg, ":", /* filter_duplicates = */ true);
                                 if (r < 0)
-                                        return log_oom();
+                                        return log_error_errno(r, "Failed to parse -s/--service= argument: %m");
                         }
 
                         break;

diff --git a/src/vmspawn/vmspawn.c b/src/vmspawn/vmspawn.c
index 15418f074231a086907f0b5554a2f52f24fe21ec..7474b301f39979220a72c50e45ba3510485acc70 100644 (file)
--- a/src/vmspawn/vmspawn.c
+++ b/src/vmspawn/vmspawn.c
@@ -2088,14 +2088,11 @@ static int run_virtual_machine(int kvm_device_fd, int vhost_device_fd) {
 
         const char *e = secure_getenv("SYSTEMD_VMSPAWN_QEMU_EXTRA");
         if (e) {
-                _cleanup_strv_free_ char **extra = NULL;
-
-                r = strv_split_full(&extra, e, /* separator= */ NULL, EXTRACT_CUNESCAPE|EXTRACT_UNQUOTE);
+                r = strv_split_and_extend_full(&cmdline, e,
+                                               /* separator = */ NULL, /* filter_duplicates = */ false,
+                                               EXTRACT_CUNESCAPE|EXTRACT_UNQUOTE);
                 if (r < 0)
-                        return log_error_errno(r, "Failed to split $SYSTEMD_VMSPAWN_QEMU_EXTRA environment variable: %m");
-
-                if (strv_extend_strv(&cmdline, extra, /* filter_duplicates= */ false) < 0)
-                        return log_oom();
+                        return log_error_errno(r, "Failed to parse $SYSTEMD_VMSPAWN_QEMU_EXTRA: %m");
         }
 
         if (DEBUG_LOGGING) {