env-util: modernize env_name_is_valid_n() a bit

If the size is specified as SIZE_MAX, then imply strlen().

diff --git a/src/basic/env-util.c b/src/basic/env-util.c
index 5a933d7f08faec8a768eb58f3c5408581b3532c7..128102c688a7cb51a14bad7944bd1fdafd87e1bf 100644 (file)
--- a/src/basic/env-util.c
+++ b/src/basic/env-util.c
@@ -26,20 +26,21 @@
         "_"
 
 static bool env_name_is_valid_n(const char *e, size_t n) {
-        if (!e)
-                return false;
+
+        if (n == SIZE_MAX)
+                n = strlen_ptr(e);
 
         if (n <= 0)
                 return false;
 
+        assert(e);
+
         if (ascii_isdigit(e[0]))
                 return false;
 
-        /* POSIX says the overall size of the environment block cannot
-         * be > ARG_MAX, an individual assignment hence cannot be
-         * either. Discounting the equal sign and trailing NUL this
-         * hence leaves ARG_MAX-2 as longest possible variable
-         * name. */
+        /* POSIX says the overall size of the environment block cannot be > ARG_MAX, an individual assignment
+         * hence cannot be either. Discounting the equal sign and trailing NUL this hence leaves ARG_MAX-2 as
+         * longest possible variable name. */
         if (n > (size_t) sysconf(_SC_ARG_MAX) - 2)
                 return false;