microsoft / onefuzz

A self-hosted Fuzzing-As-A-Service platform
MIT License
2.82k stars 198 forks source link

Include all loader errors when failing #3509

Closed Porges closed 11 months ago

Porges commented 1 year ago

Closes #2736.

Include all errors from the loader transcript when looking for missing DLLs. This will help users to analyze the problem, as this information can include missing symbols, etc.

codecov-commenter commented 1 year ago

Codecov Report

Merging #3509 (3322f45) into main (2f2a649) will decrease coverage by 0.02%. The diff coverage is 25.71%.

@@            Coverage Diff             @@
##             main    #3509      +/-   ##
==========================================
- Coverage   39.06%   39.04%   -0.02%     
==========================================
  Files         302      302              
  Lines       36904    36916      +12     
==========================================
  Hits        14415    14415              
- Misses      22489    22501      +12     
Files Coverage Δ
src/agent/dynamic-library/src/linux.rs 87.93% <80.00%> (+0.14%) :arrow_up:
src/agent/onefuzz/src/libfuzzer.rs 78.50% <62.50%> (-0.20%) :arrow_down:
src/agent/dynamic-library/src/windows.rs 18.07% <0.00%> (-0.75%) :arrow_down:
...c/agent/dynamic-library/src/bin/dynamic-library.rs 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes