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

Broken woff2 build #828

Closed
Dor1s opened this issue Sep 6, 2017 · 2 comments
Closed

Broken woff2 build #828

Dor1s opened this issue Sep 6, 2017 · 2 comments
Assignees

Comments

@Dor1s
Copy link
Contributor

Dor1s commented Sep 6, 2017

https://oss-fuzz-build-logs.storage.googleapis.com/index.html

Step #40: ++ sancov -print-coverage-pcs /workspace/out/address/convert_woff2ttf_fuzzer_new_entry
Step #40: ++ wc -l
Step #40: + local NUMBER_OF_EDGES=0
Step #40: + ((  0 < 90  ))
Step #40: + echo 'BAD BUILD: the code does not seem to have coverage instrumentation.'
Step #40: + CHECK_FAILED=1
Step #40: + ((  1 > 0  ))
Step #40: + exit 1
Step #40: BAD BUILD: the code does not seem to have coverage instrumentation.
Dor1s added a commit that referenced this issue Sep 6, 2017
…t fixed.

Need to take a closer look at the following issues:
- #815
- #816
- #817
- #818
- #819
- #820
- #821
- #822
- #823
- #824
- #825
- #826
- #827
- #828
- #829
- #830
oliverchang pushed a commit that referenced this issue Sep 7, 2017
…t fixed. (#831)

Need to take a closer look at the following issues:
- #815
- #816
- #817
- #818
- #819
- #820
- #821
- #822
- #823
- #824
- #825
- #826
- #827
- #828
- #829
- #830
@Dor1s
Copy link
Contributor Author

Dor1s commented Sep 7, 2017

Issue with a honggfuzz build (the same as #829). I guess I should disable bad build check for non-libFuzzer builds.

@Dor1s
Copy link
Contributor Author

Dor1s commented Sep 7, 2017

As honggfuzz is not fully integrated at this point, we shouldn't mark a build as broken if only honggfuzz version has some issues. Will be skipping bad build checks for honggfuzz.

@Dor1s Dor1s closed this as completed Sep 7, 2017
@Dor1s Dor1s removed the buggy target label Sep 7, 2017
Dor1s added a commit that referenced this issue Sep 8, 2017
@Dor1s Dor1s self-assigned this Sep 9, 2017
tmatth pushed a commit to tmatth/oss-fuzz that referenced this issue Oct 22, 2018
tmatth pushed a commit to tmatth/oss-fuzz that referenced this issue Oct 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant