Skip to content

Commit 64d15d6

Browse files
authored
C++: fix inc.qhelp files and change notes
1 parent 7b4300e commit 64d15d6

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

cpp/ql/src/Security/CWE/CWE-120/OverrunWriteRecommendation.inc.qhelp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,7 @@
22
"-//Semmle//qhelp//EN"
33
"qhelp.dtd">
44
<qhelp>
5-
<fragment>
65
<recommendation>
76
<p>Always control the length of buffer copy and buffer write operations. <code>strncpy</code> should be used over <code>strcpy</code>, <code>snprintf</code> over <code>sprintf</code>, and in other cases 'n-variant' functions should be preferred.</p>
87
</recommendation>
9-
</fragment>
108
</qhelp>

cpp/ql/src/Security/CWE/CWE-120/OverrunWriteReferences.inc.qhelp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
"-//Semmle//qhelp//EN"
33
"qhelp.dtd">
44
<qhelp>
5-
<fragment>
65
<references>
76

87
<li>CERT C Coding Standard: <a href="https://www.securecoding.cert.org/confluence/display/c/STR31-C.+Guarantee+that+storage+for+strings+has+sufficient+space+for+character+data+and+the+null+terminator">STR31-C. Guarantee that storage for strings has sufficient space for character data and the null terminator</a>.</li>
@@ -12,5 +11,4 @@
1211
-->
1312

1413
</references>
15-
</fragment>
1614
</qhelp>
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,4 @@
1-
lgtm,codescanning
1+
---
2+
category: newQuery
3+
---
24
* A new `cpp/very-likely-overruning-write` query has been added to the default query suite for C/C++. The query reports some results that were formerly flagged by `cpp/overruning-write`.

0 commit comments

Comments
 (0)