Skip to content

Commit e34bcef

Browse files
committed
Ruby: Move path summary visibility code into flowCheckNodeSpecific
1 parent ee185ae commit e34bcef

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

ruby/ql/lib/codeql/ruby/dataflow/internal/DataFlowPrivate.qll

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1290,16 +1290,16 @@ private import PostUpdateNodes
12901290

12911291
/** A node that performs a type cast. */
12921292
class CastNode extends Node {
1293-
CastNode() {
1294-
// ensure that all variable assignments are included in the path graph
1295-
this.(SsaDefinitionExtNode).getDefinitionExt() instanceof Ssa::WriteDefinition
1296-
}
1293+
CastNode() { none() }
12971294
}
12981295

12991296
/**
13001297
* Holds if `n` should be a FlowCheckNode, which will appear in path summaries.
13011298
*/
1302-
predicate flowCheckNodeSpecific(Node n) { none() }
1299+
predicate flowCheckNodeSpecific(Node n) {
1300+
// ensure that all variable assignments are included in the path graph
1301+
n.(SsaDefinitionExtNode).getDefinitionExt() instanceof Ssa::WriteDefinition
1302+
}
13031303

13041304
class DataFlowExpr = CfgNodes::ExprCfgNode;
13051305

0 commit comments

Comments
 (0)