Skip to content

Commit f6cddc9

Browse files
authored
Merge pull request #20678 from aschackmull/shared/rename-cfg-reach
Shared/Java: Rename ControlFlowReachability library.
2 parents 8a3f62b + 20147cd commit f6cddc9

File tree

5 files changed

+9
-9
lines changed

5 files changed

+9
-9
lines changed

csharp/ql/lib/semmle/code/csharp/controlflow/ControlFlowReachability.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
*/
44

55
import csharp
6-
private import codeql.controlflow.ControlFlow
6+
private import codeql.controlflow.ControlFlowReachability
77
private import semmle.code.csharp.controlflow.BasicBlocks
88
private import semmle.code.csharp.controlflow.Guards as Guards
99
private import semmle.code.csharp.ExprOrStmtParent

csharp/ql/lib/semmle/code/csharp/dataflow/Nullness.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ private import semmle.code.csharp.controlflow.Guards::AbstractValues
2525
private import semmle.code.csharp.dataflow.internal.SsaImpl as SsaImpl
2626
private import semmle.code.csharp.frameworks.System
2727
private import semmle.code.csharp.frameworks.Test
28-
private import semmle.code.csharp.controlflow.ControlFlowReachability as Cf
28+
private import semmle.code.csharp.controlflow.ControlFlowReachability
2929

3030
private Expr maybeNullExpr(Expr reason) {
3131
G::Internal::nullValue(result) and reason = result
@@ -260,7 +260,7 @@ private predicate defReaches(Ssa::Definition def, ControlFlow::Node cfn) {
260260
)
261261
}
262262

263-
private module NullnessConfig implements Cf::ControlFlowReachability::ConfigSig {
263+
private module NullnessConfig implements ControlFlowReachability::ConfigSig {
264264
predicate source(ControlFlow::Node node, Ssa::Definition def) { defMaybeNull(def, node, _, _) }
265265

266266
predicate sink(ControlFlow::Node node, Ssa::Definition def) {
@@ -275,7 +275,7 @@ private module NullnessConfig implements Cf::ControlFlowReachability::ConfigSig
275275
predicate uncertainFlow() { none() }
276276
}
277277

278-
private module NullnessFlow = Cf::ControlFlowReachability::Flow<NullnessConfig>;
278+
private module NullnessFlow = ControlFlowReachability::Flow<NullnessConfig>;
279279

280280
predicate maybeNullDeref(Dereference d, Ssa::SourceVariable v, string msg, Element reason) {
281281
exists(

java/ql/lib/semmle/code/java/controlflow/ControlFlow.qll renamed to java/ql/lib/semmle/code/java/controlflow/ControlFlowReachability.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ overlay[local?]
55
module;
66

77
import java
8-
private import codeql.controlflow.ControlFlow
8+
private import codeql.controlflow.ControlFlowReachability
99
private import semmle.code.java.dataflow.SSA as SSA
1010
private import semmle.code.java.controlflow.Guards as Guards
1111

@@ -57,4 +57,4 @@ private module ControlFlowInput implements InputSig<Location, ControlFlowNode, B
5757
import Guards::Guards_v3::InternalUtil
5858
}
5959

60-
module ControlFlow = Make<Location, Cfg, ControlFlowInput>;
60+
module ControlFlowReachability = Make<Location, Cfg, ControlFlowInput>;

java/ql/lib/semmle/code/java/dataflow/Nullness.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ private import IntegerGuards
1616
private import NullGuards
1717
private import semmle.code.java.Collections
1818
private import semmle.code.java.controlflow.internal.Preconditions
19-
private import semmle.code.java.controlflow.ControlFlow as Cf
19+
private import semmle.code.java.controlflow.ControlFlowReachability
2020

2121
/** Gets an expression that may be `null`. */
2222
Expr nullExpr() { result = nullExpr(_) }
@@ -248,7 +248,7 @@ private predicate impossibleEdge(BasicBlock bb1, BasicBlock bb2) {
248248
)
249249
}
250250

251-
private module NullnessConfig implements Cf::ControlFlow::ConfigSig {
251+
private module NullnessConfig implements ControlFlowReachability::ConfigSig {
252252
predicate source(ControlFlowNode node, SsaVariable def) { varMaybeNull(def, node, _, _) }
253253

254254
predicate sink(ControlFlowNode node, SsaVariable def) { varDereference(def, _) = node }
@@ -260,7 +260,7 @@ private module NullnessConfig implements Cf::ControlFlow::ConfigSig {
260260
predicate uncertainFlow() { none() }
261261
}
262262

263-
private module NullnessFlow = Cf::ControlFlow::Flow<NullnessConfig>;
263+
private module NullnessFlow = ControlFlowReachability::Flow<NullnessConfig>;
264264

265265
/**
266266
* Holds if the dereference of `v` at `va` might be `null`.

0 commit comments

Comments
 (0)