summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2020-01-28 22:39:19 +0100
committerBram Moolenaar <Bram@vim.org>2020-01-28 22:39:19 +0100
commit07da94b0f07beb15c7e26b78837def5a91e102dc (patch)
treeef04c28af35a31a4c264ece4cd6be01c1cb3d81b
parent5b18c248d3fe4961076dbc59c960ef60c80650f0 (diff)
patch 8.2.0166: Coverity warning for using uninitialized variablev8.2.0166
Problem: Coverity warning for using uninitialized variable. Solution: Check for failure.
-rw-r--r--src/version.c2
-rw-r--r--src/vim9execute.c7
2 files changed, 8 insertions, 1 deletions
diff --git a/src/version.c b/src/version.c
index 112b22cc78..d5108d9787 100644
--- a/src/version.c
+++ b/src/version.c
@@ -743,6 +743,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 166,
+/**/
165,
/**/
164,
diff --git a/src/vim9execute.c b/src/vim9execute.c
index f8efa1d6ca..001d274110 100644
--- a/src/vim9execute.c
+++ b/src/vim9execute.c
@@ -570,7 +570,12 @@ call_def_function(
if (ga_grow(&ectx.ec_stack, 1) == FAIL)
goto failed;
- get_env_tv(&name, &optval, TRUE);
+ if (get_env_tv(&name, &optval, TRUE) == FAIL)
+ {
+ semsg(_("E1060: Invalid environment variable name: %s"),
+ iptr->isn_arg.string);
+ goto failed;
+ }
*STACK_TV_BOT(0) = optval;
++ectx.ec_stack.ga_len;
}