fuzzing: add allocator harnesses #148
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
00xc
changed the title
vsntk18
reviewed
Nov 14, 2023
Add a fuzzer for the page allocator, which will run a series of semi-random actions like allocating, freeing, writing and reading pages. Signed-off-by: Carlos López <[email protected]>
Add a new fuzzing harness for the SvsmAllocator, which will perform allocations will run a series of semi-random actions like allocating, freeing and reading memory. Unfortunately this requires access to the SvsmAllocator type, so it needs to be public. Signed-off-by: Carlos López <[email protected]>
vsntk18
reviewed
Nov 14, 2023
| inited.insert(page); | ||
| } | ||
| } | ||
| Action::AllocateFile => { |
There was a problem hiding this comment.
get_file_page() and put_file_page() can also be called for a wider coverage?
There was a problem hiding this comment.
As far as I can see they are already called when cloning and dropping a PageRef respectively. In fact have made those functions private in #149 as the reference count should not be directly manipulated.
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.
Add two fuzzing harnesses. One explicitly targets page allocation, while the other one targets the higher level
SvsmAllocatorAPI.