Skip to content

Commit acc06fa

Browse files
committed
C++: Convert cpp/incorrect-string-type-conversion test to inline expectations
1 parent 18760b4 commit acc06fa

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

cpp/ql/test/query-tests/Security/CWE/CWE-704/WcharCharConversion.cpp

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@ void Test()
1818
wchar_t *lpWchar = NULL;
1919
LPCSTR lpcstr = "b";
2020

21-
lpWchar = (LPWSTR)"a"; // BUG
22-
lpWchar = (LPWSTR)lpcstr; // BUG
21+
lpWchar = (LPWSTR)"a"; // $ Alert
22+
lpWchar = (LPWSTR)lpcstr; // $ Alert
2323

24-
lpWchar = (wchar_t*)lpChar; // BUG
24+
lpWchar = (wchar_t*)lpChar; // $ Alert
2525

26-
fconstWChar((LPCWSTR)lpChar); // BUG
27-
fWChar((LPWSTR)lpChar); // BUG
26+
fconstWChar((LPCWSTR)lpChar); // $ Alert
27+
fWChar((LPWSTR)lpChar); // $ Alert
2828

2929
lpChar = (LPSTR)"a"; // Valid
3030
lpWchar = (LPWSTR)L"a"; // Valid
@@ -79,33 +79,33 @@ void CheckedConversionFalsePositiveTest3(unsigned short flags, LPTSTR buffer)
7979
if(flags & UNICODE)
8080
lpWchar = (LPWSTR)buffer; // GOOD
8181
else
82-
lpWchar = (LPWSTR)buffer; // BUG
82+
lpWchar = (LPWSTR)buffer; // $ Alert
8383

8484
if((flags & UNICODE) == 0x8)
8585
lpWchar = (LPWSTR)buffer; // GOOD
8686
else
87-
lpWchar = (LPWSTR)buffer; // BUG
87+
lpWchar = (LPWSTR)buffer; // $ Alert
8888

8989
if((flags & UNICODE) != 0x8)
90-
lpWchar = (LPWSTR)buffer; // BUG
90+
lpWchar = (LPWSTR)buffer; // $ Alert
9191
else
9292
lpWchar = (LPWSTR)buffer; // GOOD
9393

9494
// Bad operator precedence
9595
if(flags & UNICODE == 0x8)
96-
lpWchar = (LPWSTR)buffer; // BUG
96+
lpWchar = (LPWSTR)buffer; // $ Alert
9797
else
98-
lpWchar = (LPWSTR)buffer; // BUG
98+
lpWchar = (LPWSTR)buffer; // $ Alert
9999

100100
if((flags & UNICODE) != 0)
101101
lpWchar = (LPWSTR)buffer; // GOOD
102102
else
103-
lpWchar = (LPWSTR)buffer; // BUG
103+
lpWchar = (LPWSTR)buffer; // $ Alert
104104

105105
if((flags & UNICODE) == 0)
106-
lpWchar = (LPWSTR)buffer; // BUG
106+
lpWchar = (LPWSTR)buffer; // $ Alert
107107
else
108108
lpWchar = (LPWSTR)buffer; // GOOD
109109

110-
lpWchar = (LPWSTR)buffer; // BUG
110+
lpWchar = (LPWSTR)buffer; // $ Alert
111111
}
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1-
Security/CWE/CWE-704/WcharCharConversion.ql
1+
query: Security/CWE/CWE-704/WcharCharConversion.ql
2+
postprocess: utils/test/InlineExpectationsTestQuery.ql

0 commit comments

Comments
 (0)