Skip to content

Commit 9f9a960

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 108b883 commit 9f9a960

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"
@@ -199,6 +200,14 @@ void SetHeapSnapshotNearHeapLimit(const FunctionCallbackInfo<Value>& args) {
199200
Environment* env = Environment::GetCurrent(args);
200201
uint32_t limit = args[0].As<v8::Uint32>()->Value();
201202
CHECK_GT(limit, 0);
203+
204+
std::string dir = env->options()->diagnostic_dir;
205+
if (dir.empty()) {
206+
dir = Environment::GetCwd(env->exec_path());
207+
}
208+
THROW_IF_INSUFFICIENT_PERMISSIONS(
209+
env, permission::PermissionScope::kFileSystemWrite, dir);
210+
202211
env->AddHeapSnapshotNearHeapLimitCallback();
203212
env->set_heap_snapshot_near_heap_limit(limit);
204213
}

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)