Add libfuzzer based test with #443
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Signed-off-by: Alan Jowett <alanjo@microsoft.com>
Signed-off-by: Alan Jowett <alanjo@microsoft.com>
added 2 commits
May 6, 2024 08:08
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
added 2 commits
May 6, 2024 10:49
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
hawkinsw
reviewed
May 6, 2024
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces fuzzing capabilities to the project, including a new GitHub Actions workflow, and modifies the project's build system to support fuzzing. The most significant changes include the addition of the
fuzzing.ymlworkflow, the introduction of a newubpf_fuzzerexecutable, and the modification of theubpf_vmstructure to include an instruction limit.New GitHub Actions workflow:
.github/workflows/fuzzing.yml: Added a new workflow to perform fuzzing. This workflow includes steps to build the project with fuzzing enabled, run the fuzzer, and upload any found artifacts.Modifications to existing GitHub Actions workflow:
.github/workflows/main.yml: Modified thepermissionsfield to have write access to contents, changed thepushandpull_requestbranches tomainfrom*, and added a job to use the newfuzzing.ymlworkflow. [1] [2] [3]Changes to build system:
CMakeLists.txt,cmake/options.cmake,cmake/settings.cmake: Added options to enable libfuzzer and coverage, and set compiler flags accordingly. [1] [2] [3]New fuzzer executable:
libfuzzer/CMakeLists.txt,libfuzzer/libfuzz_harness.cc: Added a new executableubpf_fuzzerthat is built when fuzzing is enabled. This executable uses libfuzzer to fuzz the uBPF virtual machine. [1] [2]Modifications to uBPF VM:
vm/inc/ubpf.h,vm/ubpf_int.h,vm/ubpf_vm.c: Modified theubpf_vmstructure to include an instruction limit, added a method to set this limit, and modified theubpf_execfunction to respect this limit. Also, the bounds check function was updated for better error handling and reporting. [1] [2] [3] [4] [5] [6]