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

[fuzz] Adding dictionary_stream_round_trip fuzzer #2140

Merged
merged 2 commits into from
May 15, 2020

Conversation

bimbashrestha
Copy link
Contributor

I can't find out why CIFuzz is failing on #2139

I'm creating a fresh pr to test if it fails here.

@bimbashrestha
Copy link
Contributor Author

Fixing the memory leak got CIFuzz to pass. Looks like CIFuzz didn't catch the ASAN failure and instead reported a passing build and a different failure. Going to file an issue with them

@terrelln
Copy link
Contributor

Looks like CIFuzz didn't catch the ASAN failure and instead reported a passing build and a different failure.

I think it caught the error, which is why it failed. But then when trying to run the "reproduce" command it ran into a bug in oss-fuzz, which "swallowed" the error message.

@Cyan4973 Cyan4973 merged commit 255e5e3 into facebook:dev May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants