Skip to content

Commit e970e67

Browse files
IlyasShabiaduh95
authored andcommitted
permission: support v8.setHeapSnapshotNearHeapLimit
Signed-off-by: ishabi <ilyasshabi94@gmail.com> PR-URL: #64808 Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Aviv Keller <me@aviv.sh>
1 parent 24c7eaf commit e970e67

2 files changed

Lines changed: 18 additions & 0 deletions

File tree

src/node_v8.cc

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
#include "node.h"
2828
#include "node_external_reference.h"
2929
#include "node_profiling.h"
30+
#include "permission/permission.h"
3031
#include "util-inl.h"
3132
#include "v8-profiler.h"
3233
#include "v8.h"
@@ -200,6 +201,14 @@ void SetHeapSnapshotNearHeapLimit(const FunctionCallbackInfo<Value>& args) {
200201
Environment* env = Environment::GetCurrent(args);
201202
uint32_t limit = args[0].As<v8::Uint32>()->Value();
202203
CHECK_GT(limit, 0);
204+
205+
std::string dir = env->options()->diagnostic_dir;
206+
if (dir.empty()) {
207+
dir = Environment::GetCwd(env->exec_path());
208+
}
209+
THROW_IF_INSUFFICIENT_PERMISSIONS(
210+
env, permission::PermissionScope::kFileSystemWrite, dir);
211+
203212
env->AddHeapSnapshotNearHeapLimitCallback();
204213
env->set_heap_snapshot_near_heap_limit(limit);
205214
}

test/parallel/test-permission-fs-write-v8.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,3 +34,12 @@ const path = require('path');
3434
permission: 'FileSystemWrite',
3535
}));
3636
}
37+
38+
{
39+
assert.throws(() => {
40+
v8.setHeapSnapshotNearHeapLimit(1);
41+
}, common.expectsError({
42+
code: 'ERR_ACCESS_DENIED',
43+
permission: 'FileSystemWrite',
44+
}));
45+
}

0 commit comments

Comments
 (0)