module: fix recently introduced coverity warning

Signed-off-by: Michael Dawson <midawson@redhat.com> PR-URL: #50843 Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · Dec 4, 2023 · d4bcdd8 · d4bcdd8
1 parent cd71838
commit d4bcdd8
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