module: fix recently introduced coverity warning

Signed-off-by: Michael Dawson <[email protected]> PR-URL: #50843 Reviewed-By: James M Snell <[email protected]>
nodejs · Dec 15, 2023 · 4eedf5e · 4eedf5e
1 parent fdd8c18
commit 4eedf5e
Showing 1 changed file with 4 additions and 0 deletions.
diff --git 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