tree-wide: some more [static] related fixes

let's add [static] where it was missing so far

Drop [static] on parameters that can be NULL.

Add an assert() around parameters that have [static] and can't be NULL
hence.

Add some "const" where it was forgotten.

diff --git a/src/core/execute.c b/src/core/execute.c
index 9c086103aec724841a33d6078ae6174f20cc8869..911c3690428ae73fbf43bccba8069a51b5ece48e 100644 (file)
--- a/src/core/execute.c
+++ b/src/core/execute.c
@@ -455,12 +455,13 @@ static int setup_input(
                 const ExecContext *context,
                 const ExecParameters *params,
                 int socket_fd,
-                int named_iofds[3]) {
+                const int named_iofds[static 3]) {
 
         ExecInput i;
 
         assert(context);
         assert(params);
+        assert(named_iofds);
 
         if (params->stdin_fd >= 0) {
                 if (dup2(params->stdin_fd, STDIN_FILENO) < 0)
@@ -570,7 +571,7 @@ static int setup_output(
                 const ExecParameters *params,
                 int fileno,
                 int socket_fd,
-                int named_iofds[3],
+                const int named_iofds[static 3],
                 const char *ident,
                 uid_t uid,
                 gid_t gid,
@@ -2721,9 +2722,7 @@ out:
 static void append_socket_pair(int *array, size_t *n, const int pair[static 2]) {
         assert(array);
         assert(n);
-
-        if (!pair)
-                return;
+        assert(pair);
 
         if (pair[0] >= 0)
                 array[(*n)++] = pair[0];
@@ -2914,7 +2913,7 @@ static int exec_child(
                 ExecRuntime *runtime,
                 DynamicCreds *dcreds,
                 int socket_fd,
-                int named_iofds[3],
+                const int named_iofds[static 3],
                 int *fds,
                 size_t n_socket_fds,
                 size_t n_storage_fds,
@@ -3782,7 +3781,7 @@ static int exec_child(
 }
 
 static int exec_context_load_environment(const Unit *unit, const ExecContext *c, char ***l);
-static int exec_context_named_iofds(const ExecContext *c, const ExecParameters *p, int named_iofds[3]);
+static int exec_context_named_iofds(const ExecContext *c, const ExecParameters *p, int named_iofds[static 3]);
 
 int exec_spawn(Unit *unit,
                ExecCommand *command,
@@ -4110,13 +4109,18 @@ const char* exec_context_fdname(const ExecContext *c, int fd_index) {
         }
 }
 
-static int exec_context_named_iofds(const ExecContext *c, const ExecParameters *p, int named_iofds[static 3]) {
+static int exec_context_named_iofds(
+                const ExecContext *c,
+                const ExecParameters *p,
+                int named_iofds[static 3]) {
+
         size_t i, targets;
         const char* stdio_fdname[3];
         size_t n_fds;
 
         assert(c);
         assert(p);
+        assert(named_iofds);
 
         targets = (c->std_input == EXEC_INPUT_NAMED_FD) +
                   (c->std_output == EXEC_OUTPUT_NAMED_FD) +

diff --git a/src/core/namespace.c b/src/core/namespace.c
index 7aab2f7593c33fbb51bad8284612316b54e8ec6a..973b64007cf8719c87568a74c3ccbd0473ce1c92 100644 (file)
--- a/src/core/namespace.c
+++ b/src/core/namespace.c
@@ -1699,7 +1699,7 @@ int setup_tmp_dirs(const char *id, char **tmp_dir, char **var_tmp_dir) {
         return 0;
 }
 
-int setup_netns(int netns_storage_socket[static 2]) {
+int setup_netns(const int netns_storage_socket[static 2]) {
         _cleanup_close_ int netns = -1;
         int r, q;
 
@@ -1762,7 +1762,7 @@ fail:
         return r;
 }
 
-int open_netns_path(int netns_storage_socket[static 2], const char *path) {
+int open_netns_path(const int netns_storage_socket[static 2], const char *path) {
         _cleanup_close_ int netns = -1;
         int q, r;
 

diff --git a/src/core/namespace.h b/src/core/namespace.h
index 73fcb3fc82a3c5b6e4a8e8536c3c830fb0c7734c..e5cd8e5313a9e4ca2d8cd291543ff80e03b55da8 100644 (file)
--- a/src/core/namespace.h
+++ b/src/core/namespace.h
@@ -94,8 +94,8 @@ int setup_tmp_dirs(
                 char **tmp_dir,
                 char **var_tmp_dir);
 
-int setup_netns(int netns_storage_socket[static 2]);
-int open_netns_path(int netns_storage_socket[static 2], const char *path);
+int setup_netns(const int netns_storage_socket[static 2]);
+int open_netns_path(const int netns_storage_socket[static 2], const char *path);
 
 const char* protect_home_to_string(ProtectHome p) _const_;
 ProtectHome protect_home_from_string(const char *s) _pure_;

diff --git a/src/network/netdev/wireguard.c b/src/network/netdev/wireguard.c
index 2e9651ccebee878a473e253ad45132136ac74152..913ee2a0589772e1b0260a6009dceffb7741cf92 100644 (file)
--- a/src/network/netdev/wireguard.c
+++ b/src/network/netdev/wireguard.c
@@ -899,6 +899,8 @@ static int wireguard_read_key_file(const char *filename, uint8_t dest[static WG_
         if (!filename)
                 return 0;
 
+        assert(dest);
+
         r = read_full_file_full(filename, READ_FULL_FILE_SECURE | READ_FULL_FILE_UNBASE64, &key, &key_len);
         if (r < 0)
                 return r;

diff --git a/src/shared/efivars.c b/src/shared/efivars.c
index f16c2db739625598add525dd43034fcee5589095..a4a3c200ac61737d035d7298e61619fb9c0aac88 100644 (file)
--- a/src/shared/efivars.c
+++ b/src/shared/efivars.c
@@ -660,6 +660,8 @@ int efi_set_boot_order(uint16_t *order, size_t n) {
 static int boot_id_hex(const char s[static 4]) {
         int id = 0, i;
 
+        assert(s);
+
         for (i = 0; i < 4; i++)
                 if (s[i] >= '0' && s[i] <= '9')
                         id |= (s[i] - '0') << (3 - i) * 4;

diff --git a/src/shared/ethtool-util.c b/src/shared/ethtool-util.c
index d823931c76096b06a6935be54df2c94ac7901859..b0961df72ed7edf1e14afa51607cdc16a16371cb 100644 (file)
--- a/src/shared/ethtool-util.c
+++ b/src/shared/ethtool-util.c
@@ -652,6 +652,8 @@ int ethtool_set_glinksettings(
         struct ifreq ifr = {};
         int r;
 
+        assert(advertise);
+
         if (autonegotiation != AUTONEG_DISABLE && memeqzero(advertise, sizeof(uint32_t) * N_ADVERTISE)) {
                 log_info("ethtool: autonegotiation is unset or enabled, the speed and duplex are not writable.");
                 return 0;

diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c
index 6490fe6f0d1c4bc187e24183a3a154fe5a426ba6..52e3416cab7fd2d067f276b2016b10b275d42139 100644 (file)
--- a/src/systemctl/systemctl.c
+++ b/src/systemctl/systemctl.c
@@ -3021,6 +3021,8 @@ static enum action verb_to_action(const char *verb) {
 static const char** make_extra_args(const char *extra_args[static 4]) {
         size_t n = 0;
 
+        assert(extra_args);
+
         if (arg_scope != UNIT_FILE_SYSTEM)
                 extra_args[n++] = "--user";
 

diff --git a/src/udev/scsi_id/scsi_serial.c b/src/udev/scsi_id/scsi_serial.c
index 080b027d18a74850e54f3354d8742e52b6a71b21..8deb692f13c3f3f20a06e97d3aca988f60e2ce48 100644 (file)
--- a/src/udev/scsi_id/scsi_serial.c
+++ b/src/udev/scsi_id/scsi_serial.c
@@ -400,6 +400,9 @@ static int append_vendor_model(
                 const struct scsi_id_device *dev_scsi,
                 char buf[static VENDOR_LENGTH + MODEL_LENGTH]) {
 
+        assert(dev_scsi);
+        assert(buf);
+
         if (strnlen(dev_scsi->vendor, VENDOR_LENGTH) != VENDOR_LENGTH)
                 return log_debug_errno(SYNTHETIC_ERRNO(EINVAL),
                                        "%s: bad vendor string \"%s\"",

diff --git a/src/udev/udev-event.c b/src/udev/udev-event.c
index dbdb9065dcd1c730416a721c20be24a87cedf57b..2dacd78968c538a972579fdc780f8785a96bc349 100644 (file)
--- a/src/udev/udev-event.c
+++ b/src/udev/udev-event.c
@@ -156,6 +156,7 @@ static int get_subst_type(const char **str, bool strict, FormatSubstitutionType
         assert(str);
         assert(*str);
         assert(ret_type);
+        assert(ret_attr);
 
         if (*p == '$') {
                 p++;

diff --git a/src/udev/udev-rules.c b/src/udev/udev-rules.c
index 303594fa51ee8b7446acf91e57fe829166459ee6..e9a6453b6ca833617e957f73dee196b64336b33c 100644 (file)
--- a/src/udev/udev-rules.c
+++ b/src/udev/udev-rules.c
@@ -1464,6 +1464,8 @@ static int attr_subst_subdir(char attr[static UTIL_PATH_SIZE]) {
         const char *tail;
         size_t len, size;
 
+        assert(attr);
+
         tail = strstr(attr, "/*/");
         if (!tail)
             return 0;