Commit: patch 9.1.1851: memory leak in heredoc_get()

[Christian Brabandt]

patch 9.1.1851: memory leak in heredoc_get()

Commit: https://github.com/vim/vim/commit/dc75a204f187176cec038a274e7920837029a56b
Author: Lidong Yan <502024...@smail.nju.edu.cn>
Date: Sun Oct 12 15:05:11 2025 +0000

patch 9.1.1851: memory leak in heredoc_get()

Problem: memory leak in heredoc_get()
Solution: Free "str" if list_append_string() fails
(Lidong Yan).

In heredoc_get(), eval_all_expr_in_str() allocates a new string and
assign it to local variable "str". When list_append_string() failed
the memory pointed to by "str" leaks. Free "str" if list_append_string()
failed.

related: #18533

Signed-off-by: Lidong Yan <502024...@smail.nju.edu.cn>
Signed-off-by: Christian Brabandt <c...@256bit.org>

diff --git a/src/evalvars.c b/src/evalvars.c
index e529758ef..c840d872b 100644
--- a/src/evalvars.c
+++ b/src/evalvars.c
@@ -968,7 +968,11 @@ heredoc_get(exarg_T *eap, char_u *cmd, int script_get, int vim9compile)
}

if (list_append_string(l, str, -1) == FAIL)
+ {
+ if (free_str)
+ vim_free(str);
break;
+ }
if (free_str)
vim_free(str);
}
diff --git a/src/version.c b/src/version.c
index 3f6baf6bb..d46906ab2 100644
--- a/src/version.c
+++ b/src/version.c
@@ -729,6 +729,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
+/**/
+ 1851,
/**/
1850,
/**/