Skip to content

Commit 617d950

Browse files
jketemageoffw0
andauthored
Apply suggestions from code review
Co-authored-by: Geoffrey White <[email protected]>
1 parent 1c87875 commit 617d950

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

cpp/ql/lib/semmle/code/cpp/models/implementations/Strcat.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ class StrlcatFunction extends TaintFunction, ArrayFunction, SideEffectFunction {
104104
/**
105105
* Gets the index of the parameter that is the size of the copy (in characters).
106106
*/
107-
int getParamSize() { exists(this.getParameter(2)) and result = 2 }
107+
int getParamSize() { result = 2 }
108108

109109
/**
110110
* Gets the index of the parameter that is the source of the copy.

cpp/ql/lib/semmle/code/cpp/models/implementations/Strcpy.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ class StrcpyFunction extends ArrayFunction, DataFlowFunction, TaintFunction, Sid
3232
"wcsxfrm_l", // _strxfrm_l(dest, src, max_amount, locale)
3333
"_mbsnbcpy", // _mbsnbcpy(dest, src, max_amount)
3434
"stpcpy", // stpcpy(dest, src)
35-
"stpncpy", // stpcpy(dest, src, max_amount)
35+
"stpncpy", // stpncpy(dest, src, max_amount)
3636
"strlcpy" // strlcpy(dst, src, dst_size)
3737
])
3838
or

0 commit comments

Comments
 (0)