report: fix recent coverity warning

Fix warning about dereferencing null env Signed-off-by: Michael Dawson <mdawson@devrus.com>
nodejs · Jul 28, 2023 · 67e7828 · 67e7828
1 parent 8f0f17e
commit 67e7828
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/node_report.cc b/src/node_report.cc
@@ -857,9 +857,11 @@ std::string TriggerNodeReport(Isolate* isolate,
   // Determine the required report filename. In order of priority:
   //   1) supplied on API 2) configured on startup 3) default generated
   if (!name.empty()) {
-    THROW_IF_INSUFFICIENT_PERMISSIONS(
-        env, permission::PermissionScope::kFileSystemWrite, name, name);
-    // Filename was specified as API parameter.
+    if (env != nullptr) {
+      THROW_IF_INSUFFICIENT_PERMISSIONS(
+          env, permission::PermissionScope::kFileSystemWrite, name, name);
+      // Filename was specified as API parameter.
+    }
     filename = name;
   } else {
     std::string report_filename;