Skip to content

Adds a fuzzer for LoadEnvironment #34844

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
DavidKorczynski wants to merge 5 commits into from
Closed

Adds a fuzzer for LoadEnvironment #34844

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
1570339
build: adds a new fuzzer that targets node::LoadEnvironment()
DavidKorczynski Aug 19, 2020
4df0816
build: rename cc file to header file.
DavidKorczynski Aug 19, 2020
2a6cda5
build: Resolves the comments from bnoordhuis.
DavidKorczynski Aug 20, 2020
9735bfb
build: Fix remaining curly-bracket notation.
DavidKorczynski Aug 20, 2020
3db21a0
build: Fix a typecast so the compiler wont complain.
DavidKorczynski Aug 20, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
build: Fix remaining curly-bracket notation.
  • Loading branch information
@DavidKorczynski
DavidKorczynski committed Aug 20, 2020
commit 9735bfbe8db89bff8206150f00736ad1ed0b1454
12 changes: 4 additions & 8 deletions test/fuzzers/fuzz_env.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,15 +55,13 @@ class FuzzerFixtureHelper {

FuzzerFixtureHelper()
: allocator(ArrayBufferUniquePtr(node::CreateArrayBufferAllocator(),
&node::FreeArrayBufferAllocator))
{
&node::FreeArrayBufferAllocator)) {
isolate_ = NewIsolate(allocator.get(), &current_loop, platform.get());
CHECK_NOT_NULL(isolate_);
isolate_->Enter();
};

void Teardown()
{
void Teardown() {
platform->DrainTasks(isolate_);
isolate_->Exit();
platform->UnregisterIsolate(isolate_);
Expand All @@ -72,8 +70,7 @@ class FuzzerFixtureHelper {
}
};

void EnvTest(v8::Isolate* isolate_, char* env_string)
{
void EnvTest(v8::Isolate* isolate_, char* env_string) {
const v8::HandleScope handle_scope(isolate_);
Argv argv;

Expand All @@ -100,8 +97,7 @@ void EnvTest(v8::Isolate* isolate_, char* env_string)
context_->Exit();
}
Comment thread
bnoordhuis marked this conversation as resolved.
Resolved

extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data2, size_t size)
{
extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data2, size_t size) {
FuzzerFixtureHelper ffh;
std::string s(reinterpret_cast<const char*>(data2), size);
EnvTest(ffh.isolate_, s.c_str());
Expand Down