Skip to content

Bump LLVM version and fix aflpp version and apply patches#120

Open
tokatoka wants to merge 1 commit into
oss-garage:masterfrom
tokatoka:fix_aflpp_ver
Open

Bump LLVM version and fix aflpp version and apply patches#120
tokatoka wants to merge 1 commit into
oss-garage:masterfrom
tokatoka:fix_aflpp_ver

Conversation

@tokatoka

Copy link
Copy Markdown
Contributor

Right now, we cannot use clang-20 and future versions because AFL++ forces to add "-flto" flag when there is "-fcf-protection"
bitcoin/bitcoin#31913 (comment)

we can

  1. fix the version of AFL++ that we use, and
  2. patch AFL++ before building the docker images
    to mitigate this issue.

@dergoegge

Copy link
Copy Markdown
Member

Have you tried using -O1. I'd prefer not having to patch afl++

@tokatoka

Copy link
Copy Markdown
Contributor Author

Have you tried using -O1. I'd prefer not having to patch afl++

it doesn't work either

the real problem in AFL++ is here
https://github.com/AFLplusplus/AFLplusplus/blob/afbcb07e7602791390adfc63932efcd14d39bab8/instrumentation/SanitizerCoveragePCGUARD.so.cc#L234
The compiler pass just don't instrument anything at all if we are using -flto (which we are forced to, because -fcf-protection always comes with -flto in AFL++ without patch

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