Skip to content

Commit 0f82370

Browse files
committed
rename getHighLight() -> getAlertLocation()
1 parent 2b17242 commit 0f82370

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

javascript/ql/src/Security/CWE-078/UnsafeShellCommandConstruction.ql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,5 @@ import DataFlow::PathGraph
1818

1919
from Configuration cfg, DataFlow::PathNode source, DataFlow::PathNode sink, Sink sinkNode
2020
where cfg.hasFlowPath(source, sink) and sinkNode = sink.getNode()
21-
select sinkNode.getHighLight(), source, sink, "$@ based on libary input is later used in $@.",
22-
sinkNode.getHighLight(), sinkNode.getSinkType(), sinkNode.getCommandExecution(), "shell command"
21+
select sinkNode.getAlertLocation(), source, sink, "$@ based on libary input is later used in $@.",
22+
sinkNode.getAlertLocation(), sinkNode.getSinkType(), sinkNode.getCommandExecution(), "shell command"

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ module UnsafeShellCommandConstruction {
3636
* Gets the node that should be highlighted for this sink.
3737
* E.g. for a string concatenation, the sink is one of the leaves and the highlight is the concatenation root.
3838
*/
39-
abstract DataFlow::Node getHighLight();
39+
abstract DataFlow::Node getAlertLocation();
4040
}
4141

4242
/**
@@ -156,7 +156,7 @@ module UnsafeShellCommandConstruction {
156156

157157
override SystemCommandExecution getCommandExecution() { result = sys }
158158

159-
override DataFlow::Node getHighLight() { result = root }
159+
override DataFlow::Node getAlertLocation() { result = root }
160160
}
161161

162162
/**
@@ -181,7 +181,7 @@ module UnsafeShellCommandConstruction {
181181

182182
override SystemCommandExecution getCommandExecution() { result = sys }
183183

184-
override DataFlow::Node getHighLight() { result = this }
184+
override DataFlow::Node getAlertLocation() { result = this }
185185
}
186186

187187
/**
@@ -203,7 +203,7 @@ module UnsafeShellCommandConstruction {
203203

204204
override SystemCommandExecution getCommandExecution() { result = sys }
205205

206-
override DataFlow::Node getHighLight() { result = this }
206+
override DataFlow::Node getAlertLocation() { result = this }
207207
}
208208

209209
/**

0 commit comments

Comments
 (0)