From 19df5cc04de93519145901d9cbc78c44d0c47c5d Mon Sep 17 00:00:00 2001
From: Bram Moolenaar <Bram@vim.org>
Date: Wed, 20 Jul 2016 22:11:06 +0200
Subject: 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 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'src/userfunc.c')

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)
-- 
cgit v1.2.1