string-util: Remove utf8.h and alloc-util.h includes

src/basic/confidential-virt.c

@@ -9,6 +9,7 @@
 #include <threads.h>
 #include <unistd.h>
 
+#include "alloc-util.h"
 #include "confidential-virt.h"
 #include "confidential-virt-fundamental.h"
 #include "errno-util.h"

src/basic/percent-util.c

@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include "alloc-util.h"
 #include "parse-util.h"
 #include "percent-util.h"
 #include "string-util.h"

src/basic/sha256.c

@@ -2,6 +2,7 @@
 
 #include <unistd.h>
 
+#include "alloc-util.h"
 #include "hexdecoct.h"
 #include "macro.h"
 #include "sha256.h"

src/basic/string-util.c

@@ -1113,6 +1113,19 @@ bool string_is_safe(const char *p) {
         return true;
 }
 
+bool string_is_safe_ascii(const char *p) {
+        return ascii_is_valid(p) && string_is_safe(p);
+}
+
+char* str_realloc(char *p) {
+        /* Reallocate *p to actual size. Ignore failure, and return the original string on error. */
+
+        if (!p)
+                return NULL;
+
+        return realloc(p, strlen(p) + 1) ?: p;
+}
+
 char* string_erase(char *x) {
         if (!x)
                 return NULL;

src/basic/string-util.h

@@ -5,10 +5,8 @@
 #include <stddef.h>
 #include <string.h>
 
-#include "alloc-util.h"
 #include "macro.h"
 #include "string-util-fundamental.h"
-#include "utf8.h"
 
 /* What is interpreted as whitespace? */
 #define WHITESPACE          " \t\n\r"
@@ -225,9 +223,7 @@ static inline int strdup_to(char **ret, const char *src) {
 }
 
 bool string_is_safe(const char *p) _pure_;
-static inline bool string_is_safe_ascii(const char *p) {
-        return ascii_is_valid(p) && string_is_safe(p);
-}
+bool string_is_safe_ascii(const char *p) _pure_;
 
 DISABLE_WARNING_STRINGOP_TRUNCATION;
 static inline void strncpy_exact(char *buf, const char *src, size_t buf_len) {
@@ -254,15 +250,7 @@ static inline void* memory_startswith_no_case(const void *p, size_t sz, const ch
         return (uint8_t*) p + n;
 }
 
-static inline char* str_realloc(char *p) {
-        /* Reallocate *p to actual size. Ignore failure, and return the original string on error. */
-
-        if (!p)
-                return NULL;
-
-        return realloc(p, strlen(p) + 1) ?: p;
-}
-
+char* str_realloc(char *p);
 char* string_erase(char *x);
 
 int string_truncate_lines(const char *s, size_t n_lines, char **ret);

src/basic/terminal-util.c

@@ -52,6 +52,7 @@
 #include "terminal-util.h"
 #include "time-util.h"
 #include "user-util.h"
+#include "utf8.h"
 
 static volatile unsigned cached_columns = 0;
 static volatile unsigned cached_lines = 0;

src/cryptsetup/cryptsetup-tokens/cryptsetup-token-util.c

@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include "alloc-util.h"
 #include "cryptsetup-token-util.h"
 #include "string-util.h"
 

src/cryptsetup/cryptsetup-tokens/luks2-pkcs11.c

@@ -5,6 +5,7 @@
 
 #include "sd-json.h"
 
+#include "alloc-util.h"
 #include "cryptsetup-token-util.h"
 #include "escape.h"
 #include "hexdecoct.h"

src/fuzz/fuzz-varlink-idl.c

@@ -4,6 +4,7 @@
 
 #include "sd-varlink-idl.h"
 
+#include "alloc-util.h"
 #include "errno-util.h"
 #include "fd-util.h"
 #include "fuzz.h"

src/libsystemd/sd-bus/bus-convenience.c

@@ -3,6 +3,7 @@
 #include <sys/types.h>
 #include <unistd.h>
 
+#include "alloc-util.h"
 #include "bus-internal.h"
 #include "bus-message.h"
 #include "bus-signature.h"

src/locale/xkbcommon-util.c

@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include "alloc-util.h"
 #include "dlfcn-util.h"
 #include "log.h"
 #include "macro.h"

src/resolve/resolved-dnssd.c

@@ -13,6 +13,7 @@
 #include "resolved-manager.h"
 #include "specifier.h"
 #include "strv.h"
+#include "utf8.h"
 
 #define DNSSD_SERVICE_DIRS ((const char* const*) CONF_PATHS_STRV("systemd/dnssd"))
 

src/shared/coredump-util.c

@@ -2,6 +2,7 @@
 
 #include <elf.h>
 
+#include "alloc-util.h"
 #include "coredump-util.h"
 #include "extract-word.h"
 #include "fileio.h"

src/shared/daemon-util.c

@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include "alloc-util.h"
 #include "daemon-util.h"
 #include "fd-util.h"
 #include "log.h"

src/shared/factory-reset.c

@@ -2,6 +2,7 @@
 
 #include "sd-json.h"
 
+#include "alloc-util.h"
 #include "efivars.h"
 #include "env-util.h"
 #include "factory-reset.h"

src/shared/varlink-io.systemd.service.c

@@ -6,6 +6,7 @@
 #include "json-util.h"
 #include "log.h"
 #include "strv.h"
+#include "utf8.h"
 #include "varlink-io.systemd.service.h"
 
 static SD_VARLINK_DEFINE_METHOD(Ping);

src/sysupdate/sysupdated.c

@@ -33,6 +33,7 @@
 #include "socket-util.h"
 #include "string-table.h"
 #include "sysupdate-util.h"
+#include "utf8.h"
 
 #define FEATURES_DROPIN_NAME "systemd-sysupdate-enabled"
 

src/test/nss-test-util.c

@@ -4,6 +4,7 @@
 #include <stdio.h>
 #include <unistd.h>
 
+#include "alloc-util.h"
 #include "log.h"
 #include "nss-test-util.h"
 #include "string-util.h"

src/test/test-build-path.c

@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include "alloc-util.h"
 #include "build-path.h"
 #include "log.h"
 #include "string-util.h"

src/vmspawn/vmspawn.c

@@ -74,6 +74,7 @@
 #include "time-util.h"
 #include "tmpfile-util.h"
 #include "unit-name.h"
+#include "utf8.h"
 #include "vmspawn-mount.h"
 #include "vmspawn-register.h"
 #include "vmspawn-scope.h"