mirror of
https://github.com/github/codeql.git
synced 2026-04-21 23:14:03 +02:00
35 lines
971 B
Plaintext
35 lines
971 B
Plaintext
/**
|
|
* @name Implicit string concatenation in a list
|
|
* @description Omitting a comma between strings causes implicit concatenation which is confusing in a list.
|
|
* @kind problem
|
|
* @tags quality
|
|
* maintainability
|
|
* readability
|
|
* external/cwe/cwe-665
|
|
* @problem.severity warning
|
|
* @sub-severity high
|
|
* @precision high
|
|
* @id py/implicit-string-concatenation-in-list
|
|
*/
|
|
|
|
import python
|
|
|
|
predicate string_const(Expr s) {
|
|
s instanceof StringLiteral
|
|
or
|
|
string_const(s.(BinaryExpr).getLeft()) and string_const(s.(BinaryExpr).getRight())
|
|
}
|
|
|
|
from StringLiteral s
|
|
where
|
|
// Implicitly concatenated string is in a list and that list contains at least one other string.
|
|
exists(List l, Expr other |
|
|
not s = other and
|
|
l.getAnElt() = s and
|
|
l.getAnElt() = other and
|
|
string_const(other)
|
|
) and
|
|
exists(s.getAnImplicitlyConcatenatedPart()) and
|
|
not s.isParenthesized()
|
|
select s, "Implicit string concatenation. Maybe missing a comma?"
|