From: Damien Lejay <damien@lejay.be>
Date: Tue, 26 Aug 2025 15:55:14 +0000 (+0200)
Subject: patch 9.1.1691: over-allocation in ga_concat_strings()
X-Git-Tag: v9.1.1691^0
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=0a9ad34cad8e104dd8eaf4c45af007bf049104e0;p=thirdparty%2Fvim.git

patch 9.1.1691: over-allocation in ga_concat_strings()

Problem:  over-allocation in ga_concat_strings()
Solution: Fix ga_concat_strings() and only allocate n-1 separator length
          bytes (Damien Lejay).

ga_concat_strings() was adding the separator length for every item,
including the last one. Only (n - 1) separators are actually used.
This caused harmless but unnecessary overallocation.

closes: #18112

Signed-off-by: Damien Lejay <damien@lejay.be>
Signed-off-by: Christian Brabandt <cb@256bit.org>
---

diff --git a/src/alloc.c b/src/alloc.c
index 5f39085bce..386964c5b6 100644
--- a/src/alloc.c
+++ b/src/alloc.c
@@ -770,7 +770,10 @@ ga_concat_strings(garray_T *gap, char *sep)
     char_u	*p;
 
     for (i = 0; i < gap->ga_len; ++i)
-	len += (int)STRLEN(((char_u **)(gap->ga_data))[i]) + sep_len;
+	len += (int)STRLEN(((char_u **)(gap->ga_data))[i]);
+
+    if (gap->ga_len > 1)
+	len += (gap->ga_len - 1) * sep_len;
 
     s = alloc(len + 1);
     if (s == NULL)
diff --git a/src/version.c b/src/version.c
index b0ce4f3110..db3a4b7ac3 100644
--- a/src/version.c
+++ b/src/version.c
@@ -724,6 +724,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1691,
 /**/
     1690,
 /**/