From 37e2e14e3d32c6d1839159803934a5d31ad763aa Mon Sep 17 00:00:00 2001 From: Michael Dawson Date: Tue, 21 Nov 2023 22:31:09 +0000 Subject: [PATCH] module: fix recently introduced coverity warning Signed-off-by: Michael Dawson --- src/node_modules.cc | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/node_modules.cc b/src/node_modules.cc index ffe9fdf3f2ccdf..d8116bbf3a2ea6 100644 --- a/src/node_modules.cc +++ b/src/node_modules.cc @@ -153,6 +153,10 @@ const BindingData::PackageConfig* BindingData::GetPackageJSON( return throw_invalid_package_config(); } + // based on coverity using key with == derefs the raw value + // avoid derefing if its null + if (key.raw() == nullptr) continue; + if (key == "name") { // Though there is a key "name" with a corresponding value, // the value may not be a string or could be an invalid JSON string