From: Lennart Poettering <lennart@poettering.net>
Date: Wed, 15 Jan 2025 08:31:35 +0000 (+0100)
Subject: string-util: make strjoin() just a special case of strextend()
X-Git-Tag: v258-rc1~1604^2~1
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=34467ffa3c1861021293fcfadb6ebc24775b6fda;p=thirdparty%2Fsystemd.git

string-util: make strjoin() just a special case of strextend()

The functions are very similar, let's make them the same. If the first
argument to strextend() is NULL instead of extending a string we'll
allocate a fresh one and return that.
---

diff --git a/src/basic/string-util.c b/src/basic/string-util.c
index 5621ffc768f..5e4f57a5b13 100644
--- a/src/basic/string-util.c
+++ b/src/basic/string-util.c
@@ -77,38 +77,6 @@ char* strnappend(const char *s, const char *suffix, size_t b) {
         return r;
 }
 
-char* strjoin_real(const char *x, ...) {
-        va_list ap;
-        size_t l = 1;
-        char *r, *p;
-
-        va_start(ap, x);
-        for (const char *t = x; t; t = va_arg(ap, const char *)) {
-                size_t n;
-
-                n = strlen(t);
-                if (n > SIZE_MAX - l) {
-                        va_end(ap);
-                        return NULL;
-                }
-                l += n;
-        }
-        va_end(ap);
-
-        p = r = new(char, l);
-        if (!r)
-                return NULL;
-
-        va_start(ap, x);
-        for (const char *t = x; t; t = va_arg(ap, const char *))
-                p = stpcpy(p, t);
-        va_end(ap);
-
-        *p = 0;
-
-        return r;
-}
-
 char* strstrip(char *s) {
         if (!s)
                 return NULL;
@@ -843,12 +811,14 @@ char* strip_tab_ansi(char **ibuf, size_t *_isz, size_t highlight[2]) {
 }
 
 char* strextend_with_separator_internal(char **x, const char *separator, ...) {
+        _cleanup_free_ char *buffer = NULL;
         size_t f, l, l_separator;
         bool need_separator;
         char *nr, *p;
         va_list ap;
 
-        assert(x);
+        if (!x)
+                x = &buffer;
 
         l = f = strlen_ptr(*x);
 
@@ -856,8 +826,7 @@ char* strextend_with_separator_internal(char **x, const char *separator, ...) {
         l_separator = strlen_ptr(separator);
 
         va_start(ap, separator);
-        for (;;) {
-                const char *t;
+        for (const char *t;;) {
                 size_t n;
 
                 t = va_arg(ap, const char *);
@@ -906,9 +875,13 @@ char* strextend_with_separator_internal(char **x, const char *separator, ...) {
         va_end(ap);
 
         assert(p == nr + l);
-
         *p = 0;
 
+        /* If no buffer to extend was passed in return the start of the buffer */
+        if (buffer)
+                return TAKE_PTR(buffer);
+
+        /* Otherwise we extended the buffer: return the end */
         return p;
 }
 
diff --git a/src/basic/string-util.h b/src/basic/string-util.h
index a1592b6e6db..1bcb1c40e32 100644
--- a/src/basic/string-util.h
+++ b/src/basic/string-util.h
@@ -108,8 +108,7 @@ char* first_word(const char *s, const char *word) _pure_;
 
 char* strnappend(const char *s, const char *suffix, size_t length);
 
-char* strjoin_real(const char *x, ...) _sentinel_;
-#define strjoin(a, ...) strjoin_real((a), __VA_ARGS__, NULL)
+#define strjoin(a, ...) strextend_with_separator_internal(NULL, NULL, a, __VA_ARGS__, NULL)
 
 #define strjoina(a, ...)                                                \
         ({                                                              \