Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report sanitizer configuration in testcases & bug reports #233

Closed
mikea opened this issue Dec 30, 2016 · 3 comments
Closed

Report sanitizer configuration in testcases & bug reports #233

mikea opened this issue Dec 30, 2016 · 3 comments

Comments

@mikea
Copy link
Contributor

mikea commented Dec 30, 2016

Testcases and issues should report the sanitizer configuration. Ideally both as:

  • short name as in config (e.g. undefined)
  • precise flag values (flags change over time, we will eventually support custom configurations)
@inferno-chromium
Copy link
Collaborator

Example string "Sanitizer: address (ASAN)" will now be both on testcase report and on filed bug (e.g. https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=384).

We don't have information on build flags, i suggest we put these on wiki than in bug ? @kcc - wdyt ?

@mikea
Copy link
Contributor Author

mikea commented Jan 4, 2017

@inferno-chromium where does this information currently come from? Can we pass it through srcmap.json? (we can rename it into buildinfo.json).

The reason I'd like to see this hooked up all the way through is custom sanitizers configurations (e.g. for ubsan)

@inferno-chromium
Copy link
Collaborator

Currently memory tool name is derived from job name (hacky, but works on both clusterfuzz-internal and oss-fuzz since we follow a strict job name convention).

Reopened, lets discuss in detail in the weekly meeting. I think srcmap.json/buildinfo.json should work fine, we need to fix that for source link in stacktrace as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants