Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Remove function allowlist #2859

Merged
merged 6 commits into from
Feb 18, 2023
Merged

Conversation

ranweiler
Copy link
Member

Remove the standalone function allowlist from both OneFuzz and the coverage crate. This will let us avoid either a breaking version change or having to maintain tedious back-compat code when we implement #2854.

@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2023

Codecov Report

Merging #2859 (fa47bb4) into main (1ac3fd4) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2859      +/-   ##
==========================================
+ Coverage   28.08%   28.09%   +0.01%     
==========================================
  Files         302      302              
  Lines       35653    35629      -24     
==========================================
- Hits        10012    10011       -1     
+ Misses      25641    25618      -23     
Impacted Files Coverage Δ
src/ApiService/ApiService/OneFuzzTypes/Enums.cs 32.39% <ø> (ø)
src/ApiService/ApiService/OneFuzzTypes/Model.cs 69.62% <ø> (+0.04%) ⬆️
src/ApiService/ApiService/onefuzzlib/Config.cs 0.00% <ø> (ø)
src/ApiService/ApiService/onefuzzlib/Defs.cs 0.00% <ø> (ø)
src/agent/coverage/src/allowlist.rs 76.99% <ø> (+3.26%) ⬆️
src/agent/coverage/src/binary.rs 47.19% <ø> (+3.44%) ⬆️
src/agent/onefuzz-task/src/local/coverage.rs 0.00% <ø> (ø)
...c/agent/onefuzz-task/src/tasks/coverage/generic.rs 0.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ranweiler ranweiler merged commit d5cff03 into microsoft:main Feb 18, 2023
@ranweiler ranweiler deleted the remove-fn-allowlist branch February 18, 2023 04:19
@mgreisen mgreisen mentioned this pull request Mar 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants