Skip to content

Commit 0004b88

Browse files
committed
Debug query according to the new test cases
1 parent bdfc5bd commit 0004b88

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

javascript/frameworks/cap/lib/advanced_security/javascript/frameworks/cap/CAPCqlInjectionQuery.qll

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,18 @@ class CqlClauseWithStringConcatParameter instanceof CqlClause {
3131
* concatenation expression.
3232
*/
3333
class CqlShortcutMethodCallWithStringConcat instanceof CqlShortcutMethodCall {
34+
DataFlow::Node stringConcatParameter;
35+
3436
CqlShortcutMethodCallWithStringConcat() {
35-
exists(StringConcatenation::getAnOperand(super.getAQueryParameter()))
37+
stringConcatParameter = super.getAQueryParameter() and
38+
exists(StringConcatenation::getAnOperand(stringConcatParameter))
3639
}
3740

3841
Location getLocation() { result = super.getLocation() }
3942

4043
string toString() { result = super.toString() }
44+
45+
DataFlow::Node getStringConcatParameter() { result = stringConcatParameter }
4146
}
4247

4348
/**
@@ -66,7 +71,7 @@ class CqlInjectionConfiguration extends TaintTracking::Configuration {
6671
)
6772
or
6873
exists(CqlShortcutMethodCallWithStringConcat queryRunnerCall |
69-
node = queryRunnerCall.(CqlQueryRunnerCall).getAQueryParameter()
74+
node = queryRunnerCall.getStringConcatParameter()
7075
)
7176
or
7277
exists(AwaitExpr await, CqlClauseWithStringConcatParameter cqlClauseWithStringConcat |

0 commit comments

Comments
 (0)