Hi, I found a false negative in Cppcheck 2.21.0 when a logical expression selects a zero-valued ternary branch.
Affected tool
Cppcheck 2.21.0
Affected checker
Cppcheck CheckOther::checkZeroDivision (zerodiv / zerodivcond)
Minimal reproducer
int divide_after_logical_ternary(bool left, bool right) { int divisor = (left || right) ? 0 : 1; return 1 / divisor; }
Reproduction command
cppcheck --version cppcheck --enable=warning --std=c++11 --checkers-report=checkers.txt --template='{file}:{line}:{column}: [{id}] {message}' cppcheck-zerodiv-fn-logical-ternary.cpp
Current behavior
Cppcheck produces neither a zerodiv nor a zerodivcond diagnostic. The checker report lists CheckOther::checkZeroDivision as active.
zerodiv
zerodivcond
CheckOther::checkZeroDivision
Expected behavior
Cppcheck should report a division-by-zero diagnostic at line 3 because either true operand selects the branch that assigns zero to divisor.
divisor
Thanks for reporting, ticket is here: https://trac.cppcheck.net/ticket/15052
Log in to post a comment.
Hi, I found a false negative in Cppcheck 2.21.0 when a logical expression selects a zero-valued ternary branch.
Affected tool
Cppcheck 2.21.0Affected checker
Cppcheck CheckOther::checkZeroDivision (zerodiv / zerodivcond)Minimal reproducer
Reproduction command
Current behavior
Cppcheck produces neither a
zerodivnor azerodivconddiagnostic. The checker report listsCheckOther::checkZeroDivisionas active.Expected behavior
Cppcheck should report a division-by-zero diagnostic at line 3 because either true operand selects the branch that assigns zero to
divisor.Thanks for reporting, ticket is here: https://trac.cppcheck.net/ticket/15052