remove duplicate reflectiveCallNode method, and removing redundant getExpr() method

erik-krogh · erik-krogh · commit 538690eee637 · 2019-11-13T15:53:21.000+01:00
diff --git a/javascript/ql/src/semmle/javascript/dataflow/Configuration.qll b/javascript/ql/src/semmle/javascript/dataflow/Configuration.qll
@@ -308,7 +308,7 @@ abstract class BarrierGuardNode extends DataFlow::Node {
     exists(SsaRefinementNode ref, boolean outcome |
       nd = DataFlow::ssaDefinitionNode(ref) and
       forex(SsaVariable input | input = ref.getAnInput() |
-        getExpr() = ref.getGuard().getTest() and
+        getEnclosingExpr() = ref.getGuard().getTest() and
         outcome = ref.getGuard().(ConditionGuardNode).getOutcome() and
         barrierGuardBlocksExpr(this, outcome, input.getAUse(), label)
       )
@@ -317,20 +317,13 @@ abstract class BarrierGuardNode extends DataFlow::Node {
     // 2) `nd` is an instance of an access path `p`, and dominated by a barrier for `p`
     exists(AccessPath p, BasicBlock bb, ConditionGuardNode cond, boolean outcome |
       nd = DataFlow::valueNode(p.getAnInstanceIn(bb)) and
-      getExpr() = cond.getTest() and
+      getEnclosingExpr() = cond.getTest() and
       outcome = cond.getOutcome() and
       barrierGuardBlocksAccessPath(this, outcome, p, label) and
       cond.dominates(bb)
     )
   }
 
-  /** Gets the corresponding expression, including that of reflective calls. */
-  private Expr getExpr() {
-    result = asExpr()
-    or
-    this = DataFlow::reflectiveCallNode(result)
-  }
-
   /**
    * Holds if this node blocks expression `e` provided it evaluates to `outcome`.
    *
diff --git a/javascript/ql/src/semmle/javascript/dataflow/DataFlow.qll b/javascript/ql/src/semmle/javascript/dataflow/DataFlow.qll
@@ -996,16 +996,6 @@ module DataFlow {
    */
   DataFlow::Node globalAccessPathRootPseudoNode() { result instanceof TGlobalAccessPathRoot }
   
-  /**
-   * Gets a data flow node representing the underlying call performed by the given
-   * call to `Function.prototype.call` or `Function.prototype.apply`.
-   *
-   * For example, for an expression `fn.call(x, y)`, this gets a call node with `fn` as the
-   * callee, `x` as the receiver, and `y` as the first argument.
-   */
-  DataFlow::InvokeNode reflectiveCallNode(InvokeExpr expr) { result = TReflectiveCallNode(expr, _) }
-  
-
   /**
    * Gets a data flow node representing the underlying call performed by the given
    * call to `Function.prototype.call` or `Function.prototype.apply`.
