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 wolfssl build #829

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

Broken wolfssl build #829

Dor1s opened this issue Sep 6, 2017 · 5 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/pem_cert
Step #40: ++ wc -l
Step #40: BAD BUILD: the code does not seem to have coverage instrumentation.
Step #40: + local NUMBER_OF_EDGES=1
Step #40: + ((  1 < 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
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
@JacobBarthelmeh
Copy link
Contributor

Hi Dor1s is there an action wolfSSL should take with this? What does the label buggy target mean?

@Dor1s
Copy link
Contributor Author

Dor1s commented Sep 7, 2017

Hi @JacobBarthelmeh, thanks for checking. I don't think that woflSSL should do anything right now, I need to manually verify this particular issue as well as some others.

buggy target label means that the root cause seems to be inside of the target / project, not on OSS-Fuzz side.

@Dor1s
Copy link
Contributor Author

Dor1s commented Sep 7, 2017

There is an issue when pem_cert fuzz target is being built with hongffuzz. When built with libFuzzer, it's absolutely fine.

@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
@JacobBarthelmeh
Copy link
Contributor

Thanks for the update Dor1s.

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

2 participants