Skip to content

Commit 7d491af

Browse files
committed
C++: More test cases.
1 parent b7d2e0c commit 7d491af

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

cpp/ql/test/query-tests/Likely Bugs/Likely Typos/inconsistentLoopDirection/inconsistentLoopDirection.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ void FalseNegativeTestCases()
179179
for (int i = 100; i > 0; i ++ ) {} // BUG
180180
}
181181

182-
void IntendedOverflow()
182+
void IntendedOverflow(unsigned char p)
183183
{
184184
const unsigned char m = 10;
185185
unsigned char i;
@@ -195,4 +195,7 @@ void IntendedOverflow()
195195

196196
for (s = 63; s < 64; s--) {} // BAD (signed numbers don't wrap at 0 / at all)
197197
for (s = m + 1; s < m; s--) {} // BAD (never runs)
198+
199+
for (i = p - 1; i < p; i--) {} // GOOD
200+
for (s = p - 1; s < p; s--) {} // BAD [NOT DETECTED]
198201
}

0 commit comments

Comments
 (0)