mirror of
https://github.com/github/codeql.git
synced 2026-04-29 18:55:14 +02:00
Apply suggestions from code review
Co-authored-by: Taus <tausbn@github.com>
This commit is contained in:
committed by
Rasmus Lerchedahl Petersen
parent
4d856d4461
commit
63b732ce1f
@@ -141,7 +141,7 @@ class Node extends TNode {
|
||||
LocalSourceNode getALocalSource() { result.flowsTo(this) }
|
||||
|
||||
/**
|
||||
* Gets a local source node from which data may flow to this node in zero or more local steps.
|
||||
* Gets a local source node from which data may flow to this node in zero or more local taint-flow steps.
|
||||
*/
|
||||
LocalSourceNode getALocalTaintSource() { result.taintFlowsTo(this) }
|
||||
}
|
||||
|
||||
@@ -71,7 +71,7 @@ private module Cached {
|
||||
*/
|
||||
cached
|
||||
predicate hasLocalSource(Node sink, Node source) {
|
||||
// Declaring `source` to be a `SourceNode` currently causes a redundant check in the
|
||||
// Declaring `source` to be a `LocalSourceNode` currently causes a redundant check in the
|
||||
// recursive case, so instead we check it explicitly here.
|
||||
source = sink and
|
||||
source instanceof LocalSourceNode
|
||||
@@ -89,7 +89,7 @@ private module Cached {
|
||||
*/
|
||||
cached
|
||||
predicate hasLocalTaintSource(Node sink, Node source) {
|
||||
// Declaring `source` to be a `SourceNode` currently causes a redundant check in the
|
||||
// Declaring `source` to be a `LocalSourceNode` currently causes a redundant check in the
|
||||
// recursive case, so instead we check it explicitly here.
|
||||
source = sink and
|
||||
source instanceof LocalSourceNode
|
||||
|
||||
Reference in New Issue
Block a user