test,systemctl,nspawn: use "const char*" instead of "char*" as iterator for FOREACH_S...

The macro iterates through literal strings (i.e. constant strings),
hence it's more correct to have the iterator const too.

diff --git a/src/nspawn/nspawn.c b/src/nspawn/nspawn.c
index 91c97b60a7154f348e918f5e4fdf88c5a740b266..8e81f40e0db50b8286dd99b7ca8b999f9d518127 100644 (file)
--- a/src/nspawn/nspawn.c
+++ b/src/nspawn/nspawn.c
@@ -1964,11 +1964,11 @@ static int setup_hostname(void) {
 }
 
 static int setup_journal(const char *directory) {
-        sd_id128_t this_id;
         _cleanup_free_ char *d = NULL;
-        const char *p, *q;
+        const char *dirname, *p, *q;
+        sd_id128_t this_id;
+        char id[33];
         bool try;
-        char id[33], *dirname;
         int r;
 
         /* Don't link journals in ephemeral mode */

diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c
index 629f9cb505f3a7a7312bb4092ed1a8f057a0c6d7..63dae2c87285667bc833206bacb83989b4d22fe1 100644 (file)
--- a/src/systemctl/systemctl.c
+++ b/src/systemctl/systemctl.c
@@ -6933,9 +6933,9 @@ static int run_editor(char **paths) {
         if (r < 0)
                 return r;
         if (r == 0) {
-                char **editor_args = NULL, **tmp_path, **original_path, *p;
+                char **editor_args = NULL, **tmp_path, **original_path;
                 size_t n_editor_args = 0, i = 1, argc;
-                const char **args, *editor;
+                const char **args, *editor, *p;
 
                 argc = strv_length(paths)/2 + 1;
 

diff --git a/src/test/test-execute.c b/src/test/test-execute.c
index cea68b8cfe5af9f68b63e16a18e31297f4c01f96..4c240fffdae1060a2df68b5f17c1d2844ca267bd 100644 (file)
--- a/src/test/test-execute.c
+++ b/src/test/test-execute.c
@@ -131,7 +131,7 @@ static bool check_user_has_group_with_same_name(const char *name) {
 }
 
 static bool is_inaccessible_available(void) {
-        char *p;
+        const char *p;
 
         FOREACH_STRING(p,
                 "/run/systemd/inaccessible/reg",