Skip to content

Commit 67e7828

Browse files
committed
report: fix recent coverity warning
Fix warning about dereferencing null env Signed-off-by: Michael Dawson <mdawson@devrus.com>
1 parent 8f0f17e commit 67e7828

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/node_report.cc

+5-3
Original file line numberDiff line numberDiff line change
@@ -857,9 +857,11 @@ std::string TriggerNodeReport(Isolate* isolate,
857857
// Determine the required report filename. In order of priority:
858858
// 1) supplied on API 2) configured on startup 3) default generated
859859
if (!name.empty()) {
860-
THROW_IF_INSUFFICIENT_PERMISSIONS(
861-
env, permission::PermissionScope::kFileSystemWrite, name, name);
862-
// Filename was specified as API parameter.
860+
if (env != nullptr) {
861+
THROW_IF_INSUFFICIENT_PERMISSIONS(
862+
env, permission::PermissionScope::kFileSystemWrite, name, name);
863+
// Filename was specified as API parameter.
864+
}
863865
filename = name;
864866
} else {
865867
std::string report_filename;

0 commit comments

Comments
 (0)