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

parse stderr for ASAN for libfuzzer targets#127

Merged
bmc-msft merged 8 commits into
microsoft:mainfrom
bmc-msft:non-asan-libfuzzer-crashes
Oct 9, 2020
Merged

parse stderr for ASAN for libfuzzer targets#127
bmc-msft merged 8 commits into
microsoft:mainfrom
bmc-msft:non-asan-libfuzzer-crashes

Conversation

@bmc-msft

@bmc-msft bmc-msft commented Oct 9, 2020

Copy link
Copy Markdown
Contributor

Summary of the Pull Request

Adds parsing stderr to look for ASAN style logs during libfuzzer crash reporting

PR Checklist

  • Applies to work item: Go language support #118
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

This PR does two things:

  1. Adds parsing of stderr to libfuzzer targets
  2. Standardizes run_cmd onto our own Output

This doesn't implement stderr parsing for non-libfuzzer tasks, nor does it attempt to parse the output if you used a debugger. Eventually, it would be good to normalize our debugger wrappers to include the same Output but that is out of scope for this PR.

Validation Steps Performed

  1. Run standard libfuzzer binary and see crash reports
  2. Run a test with a go-fuzz based binary and see crash reports

@bmc-msft bmc-msft linked an issue Oct 9, 2020 that may be closed by this pull request
@bmc-msft bmc-msft mentioned this pull request Oct 9, 2020
@bmc-msft bmc-msft merged commit 6e2a722 into microsoft:main Oct 9, 2020
@bmc-msft bmc-msft deleted the non-asan-libfuzzer-crashes branch October 9, 2020 21:00
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2021
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.

Go language support

3 participants