Skip to content

Commit e467d3c

Browse files
committed
use dominating write check in js/path-injection
1 parent 6bc821b commit e467d3c

File tree

3 files changed

+490
-1
lines changed

3 files changed

+490
-1
lines changed

javascript/ql/src/semmle/javascript/security/dataflow/TaintedPathCustomizations.qll

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -649,7 +649,8 @@ module TaintedPath {
649649
exists(DataFlow::PropRead read | read = dst |
650650
src = read.getBase() and
651651
read.getPropertyName() != "length" and
652-
srclabel = dstlabel
652+
srclabel = dstlabel and
653+
not AccessPath::DominatingPaths::hasDominatingWrite(read)
653654
)
654655
or
655656
// string method calls of interest

0 commit comments

Comments
 (0)