Skip to content

Commit ec9c370

Browse files
committed
address review feedback
1 parent 35a7e15 commit ec9c370

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

javascript/ql/src/semmle/javascript/dataflow/TaintTracking.qll

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -669,14 +669,14 @@ module TaintTracking {
669669
/**
670670
* Holds if the property `loadStep` should be copied from the object `pred` to the property `storeStep` of object `succ`.
671671
*
672-
* This step is used to copy a value the value of our pseudo-property that can later be accessed using a `get` or `getAll` call.
672+
* This step is used to copy the value of our pseudo-property that can later be accessed using a `get` or `getAll` call.
673673
*/
674674
override predicate loadStoreStep(DataFlow::Node pred, DataFlow::Node succ, string loadProp, string storeProp) {
675675
loadProp = hiddenUrlPseudoProperty() and
676676
storeProp = getableUrlPseudoProperty() and
677-
exists(DataFlow::PropRead write | write = succ |
678-
write.getPropertyName() = "searchParams" and
679-
write.getBase() = pred
677+
exists(DataFlow::PropRead read | read = succ |
678+
read.getPropertyName() = "searchParams" and
679+
read.getBase() = pred
680680
)
681681
}
682682

0 commit comments

Comments
 (0)