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

envoy: add http_parser external lib coverage #7637

Merged
merged 1 commit into from
Apr 29, 2022

Conversation

adisuissa
Copy link
Contributor

Envoy has introduced a temporary http_parser lib (see: envoyproxy/envoy#19749).
This PR adds the files needed for coverage.

Signed-off-by: Adi Suissa-Peleg adip@google.com

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
@adisuissa
Copy link
Contributor Author

cc @asraa
This time I've deleted the docker images and was able to repro the issue:

Step #5: error: /workspace/out/libfuzzer-coverage-x86_64/proc/self/cwd/external/envoy/bazel/external/http_parser/http_parser.c: No such file or directory
Step #5: warning: Could not read coverage for 'http_parser_execute'.

This PR should fix it.

@asraa asraa merged commit fb44849 into google:master Apr 29, 2022
MartinPetkov pushed a commit to MartinPetkov/oss-fuzz that referenced this pull request Aug 15, 2022
Signed-off-by: Adi Suissa-Peleg <adip@google.com>
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

Successfully merging this pull request may close these issues.

2 participants