Skip to content

Commit c0aae3d

Browse files
tiferetStephan Brandauer
andauthored
Apply suggestions from code review
Co-authored-by: Stephan Brandauer <[email protected]>
1 parent d17383d commit c0aae3d

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

javascript/ql/experimental/adaptivethreatmodeling/lib/experimental/adaptivethreatmodeling/ATMConfig.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ abstract class AtmConfig extends JS::TaintTracking::Configuration {
159159
* Holds if if `sink` is an effective sink with flow from `source` which gets used as a sink candidate for scoring
160160
* with the ML model.
161161
*/
162-
predicate isSinkCandidate(JS::DataFlow::PathNode sink) {
162+
predicate isSinkCandidateWithFlow(JS::DataFlow::Node sink) {
163163
exists(JS::DataFlow::PathNode source |
164164
this.hasFlowPath(source, sink) and
165165
not AtmResultsInfo::isFlowLikelyInBaseQuery(source.getNode(), sink.getNode())

javascript/ql/experimental/adaptivethreatmodeling/test/endpoint_large_scale/ExtractEndpointDataInference.ql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,5 +23,5 @@ private import experimental.adaptivethreatmodeling.XssThroughDomATM as XssThroug
2323
query predicate isSinkCandidateForQuery(
2424
AtmConfig::AtmConfig queryConfig, JS::DataFlow::PathNode sink
2525
) {
26-
queryConfig.isSinkCandidate(sink)
26+
queryConfig.isSinkCandidateWithFlow(sink)
2727
}

0 commit comments

Comments
 (0)