From 9ce6e363f40708c2474082c72de22f0234dd9d16 Mon Sep 17 00:00:00 2001 From: Christopher Beeson Date: Sun, 26 Apr 2020 18:28:27 -0400 Subject: [PATCH] worker: fix process.env var empty key access MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR-URL: https://github.com/nodejs/node/pull/32921 Reviewed-By: Ben Noordhuis Reviewed-By: David Carlier Reviewed-By: James M Snell Reviewed-By: Franziska Hinkelmann Reviewed-By: Zeyu Yang Reviewed-By: Anna Henningsen Reviewed-By: Juan José Arboleda --- src/node_env_var.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/node_env_var.cc b/src/node_env_var.cc index d6440ba403cad4..51dde17854ca41 100644 --- a/src/node_env_var.cc +++ b/src/node_env_var.cc @@ -210,7 +210,7 @@ void MapKVStore::Set(Isolate* isolate, Local key, Local value) { Mutex::ScopedLock lock(mutex_); Utf8Value key_str(isolate, key); Utf8Value value_str(isolate, value); - if (*key_str != nullptr && *value_str != nullptr) { + if (*key_str != nullptr && key_str.length() > 0 && *value_str != nullptr) { map_[std::string(*key_str, key_str.length())] = std::string(*value_str, value_str.length()); }