From 19df5cc04de93519145901d9cbc78c44d0c47c5d Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Wed, 20 Jul 2016 22:11:06 +0200 Subject: [PATCH] patch 7.4.2083 Problem: Coverity complains about not restoring a value. Solution: Restore the value, although it's not really needed. Change return to jump to cleanup, might leak memory. --- src/userfunc.c | 5 ++++- src/version.c | 2 ++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/userfunc.c b/src/userfunc.c index 9ffafc371..ffbbc2d21 100644 --- a/src/userfunc.c +++ b/src/userfunc.c @@ -199,14 +199,17 @@ get_function_args( break; } if (newargs != NULL && ga_grow(newargs, 1) == FAIL) - return FAIL; + goto err_ret; if (newargs != NULL) { c = *p; *p = NUL; arg = vim_strsave(arg); if (arg == NULL) + { + *p = c; goto err_ret; + } /* Check for duplicate argument name. */ for (i = 0; i < newargs->ga_len; ++i) diff --git a/src/version.c b/src/version.c index acc5c33d1..69da11921 100644 --- a/src/version.c +++ b/src/version.c @@ -758,6 +758,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 2083, /**/ 2082, /**/ -- 2.50.1