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

new afl++ token/libafl experiment #1608

Merged
merged 9 commits into from
Jan 12, 2023
Merged

new afl++ token/libafl experiment #1608

merged 9 commits into from
Jan 12, 2023

Conversation

vanhauser-thc
Copy link
Collaborator

hopefully fuzzbench is not throwing up again :)

@jonathanmetzman
Copy link
Contributor

Weird that CI is actually in a good state now

@jonathanmetzman jonathanmetzman merged commit 71feb67 into google:master Jan 12, 2023
@vanhauser-thc
Copy link
Collaborator Author

I meant the state the last weeks when there were spurious build failures.

but it seems they are here again. aflplusplus, _tokens, _tokens_only all share the same commit hash, but one fails in standard, one in oss and another one in oss too.
if aflplusplus fails in oss, the other two have to as well if there would really be a bug. and _tokens and tokens_only need to fail the same tests, but one is oss and the other is standard.

@vanhauser-thc
Copy link
Collaborator Author

hmm it is again spurious checkout failures, at least for one. @alan32liu

@DonggeLiu
Copy link
Contributor

Hi @vanhauser-thc, I was investigating this, but I really did not find a possible explanation:
It seems more like a strange issue in git/docker than in your code or FuzzBench.

But I guess there are several more things we can try together, although they might not matter:

  1. Specify and only pull the belonging branch of the commit. Given the commit e4ff0eb is in dev instead of the default (stable), maybe do git clone -b dev https://github.com/AFLplusplus/AFLplusplus /afl.
  2. Reorganise the docker commands, e.g. splitting cd /afl and git checkout to two RUNs.

Again, I don't think these will somehow solve the issue, but it might avoid triggering them.
Please let me know what you think.

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.

3 participants