From 07da94b0f07beb15c7e26b78837def5a91e102dc Mon Sep 17 00:00:00 2001
From: Bram Moolenaar <Bram@vim.org>
Date: Tue, 28 Jan 2020 22:39:19 +0100
Subject: [PATCH] patch 8.2.0166: Coverity warning for using uninitialized
 variable

Problem:    Coverity warning for using uninitialized variable.
Solution:   Check for failure.
---
 src/version.c     | 2 ++
 src/vim9execute.c | 7 ++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/version.c b/src/version.c
index 112b22cc78..d5108d9787 100644
--- a/src/version.c
+++ b/src/version.c
@@ -742,6 +742,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    166,
 /**/
     165,
 /**/
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;
 		}