Mercurial > vim

changeset 31657:4473453c09b1 v9.0.1161

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
patch 9.0.1161: Coverity warns for using strcpy() Commit: https://github.com/vim/vim/commit/c8ab30a349ee45ef6c3b6af8ee5ff53d2f7bf891 Author: Bram Moolenaar <Bram@vim.org> Date: Mon Jan 9 11:35:47 2023 +0000 patch 9.0.1161: Coverity warns for using strcpy() Problem: Coverity warns for using strcpy(). Solution: Call a function to set the function name.
author Bram Moolenaar <Bram@vim.org>
date Mon, 09 Jan 2023 12:45:05 +0100
parents b92e3ea02167
children c876bc656c4b
files src/userfunc.c src/version.c
diffstat 2 files changed, 6 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/src/userfunc.c
+++ b/src/userfunc.c
@@ -2585,8 +2585,6 @@ copy_lambda_to_global_func(
 	    || ga_copy_strings(&ufunc->uf_lines, &fp->uf_lines) == FAIL)
 	goto failed;
 
-    fp->uf_name_exp = ufunc->uf_name_exp == NULL ? NULL
-					 : vim_strsave(ufunc->uf_name_exp);
     if (ufunc->uf_arg_types != NULL)
     {
 	fp->uf_arg_types = ALLOC_MULT(type_T *, fp->uf_args.ga_len);
@@ -2604,7 +2602,10 @@ copy_lambda_to_global_func(
     fp->uf_ret_type = ufunc->uf_ret_type;
 
     fp->uf_refcount = 1;
-    STRCPY(fp->uf_name, global);
+
+    fp->uf_name_exp = NULL;
+    set_ufunc_name(fp, global);
+
     hash_add(&func_hashtab, UF2HIKEY(fp), "copy lambda");
 
     // the referenced dfunc_T is now used one more time
--- a/src/version.c
+++ b/src/version.c
@@ -696,6 +696,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    1161,
+/**/
     1160,
 /**/
     1159,