Skip to content

Commit 19df5cc

Browse files
committed
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.
1 parent 397eadb commit 19df5cc

2 files changed

Lines changed: 6 additions & 1 deletion

File tree

src/userfunc.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -199,14 +199,17 @@ get_function_args(
199199
break;
200200
}
201201
if (newargs != NULL && ga_grow(newargs, 1) == FAIL)
202-
return FAIL;
202+
goto err_ret;
203203
if (newargs != NULL)
204204
{
205205
c = *p;
206206
*p = NUL;
207207
arg = vim_strsave(arg);
208208
if (arg == NULL)
209+
{
210+
*p = c;
209211
goto err_ret;
212+
}
210213

211214
/* Check for duplicate argument name. */
212215
for (i = 0; i < newargs->ga_len; ++i)

src/version.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -758,6 +758,8 @@ static char *(features[]) =
758758

759759
static int included_patches[] =
760760
{ /* Add new patch number below this line */
761+
/**/
762+
2083,
761763
/**/
762764
2082,
763765
/**/

0 commit comments

Comments
 (0)