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

Port 'missing field' error fix to C# #2905

Merged

Conversation

nharper285
Copy link
Contributor

@nharper285 nharper285 commented Mar 2, 2023

Summary of the Pull Request

What is this about?
A 'missing field' error affecting libfuzzer was previously addressed in #1739. This fix addressed #1672.

This PR ports the fix to the C# code.
Closes #2883

PR Checklist

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@codecov-commenter
Copy link

codecov-commenter commented Mar 3, 2023

Codecov Report

Merging #2905 (dd28163) into main (091c870) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2905      +/-   ##
==========================================
- Coverage   28.22%   28.21%   -0.01%     
==========================================
  Files         303      303              
  Lines       36013    36018       +5     
==========================================
  Hits        10164    10164              
- Misses      25849    25854       +5     
Impacted Files Coverage Δ
src/ApiService/ApiService/OneFuzzTypes/Model.cs 69.35% <0.00%> (-0.09%) ⬇️
src/ApiService/ApiService/onefuzzlib/Config.cs 0.00% <0.00%> (ø)
src/ApiService/ApiService/onefuzzlib/Defs.cs 0.00% <0.00%> (ø)

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

@nharper285 nharper285 merged commit 5bfcc4e into microsoft:main Mar 3, 2023
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Preserve existing outputs patch
3 participants