Fix #14918: fuzzing crash (stack-overflow) in findTokensSkipDeadCodeImpl() - #8732
Fix #14918: fuzzing crash (stack-overflow) in findTokensSkipDeadCodeImpl()#8732ludviggunne wants to merge 3 commits into
Conversation
|
FYI multiple post-in/decrements are invalid but it is valid for pre-in/decrement. |
Hm, I get this when I try it: |
I suppose it can be made to work with operator overloading. I will add a language check. |
491d30c to
15b6a74
Compare
|
Ah, it is only allowed in C++. It is also not handled by us - I filed https://trac.cppcheck.net/ticket/14920 about it. |
15b6a74 to
b78df5d
Compare
This is caused by a behavior change in CMake 4.4 (which is not rolled out on all runners yet). |
|
I filed https://trac.cppcheck.net/ticket/14922 about it. |
b78df5d to
3cbcc32
Compare
Co-authored-by: Daniel Marjamäki <[email protected]>
| syntaxError(tok, tok->strAt(1) + " " + tok->strAt(2)); | ||
| } | ||
| else if (!cpp && Token::Match(tok, "++|-- ++|--")) { | ||
| syntaxError(tok, tok->str() + tok->strAt(1)); |
There was a problem hiding this comment.
does this mean that we still crash for the fuzzer code if --language=c++ is used?
There was a problem hiding this comment.
No, we already throw a syntax error for C++.
No description provided.