changes: now it generates simple lattice without having intermediate nodes for the...
[IRC.git] / Robust / src / Analysis / SSJava / LocationInference.java
index 32b3a12409ea112b57ab25319a5e14708f479548..26bd8c29a9d8e64bd9138ad83bbeec237eac733b 100644 (file)
@@ -116,6 +116,8 @@ public class LocationInference {
   // set of callees reachable from the method
   private Map<MethodDescriptor, GlobalFlowGraph> mapMethodDescriptorToSubGlobalFlowGraph;
 
+  private Map<MethodInvokeNode, Map<NTuple<Descriptor>, NTuple<Descriptor>>> mapMethodInvokeNodeToMapCallerArgToCalleeArg;
+
   public static final String GLOBALLOC = "GLOBALLOC";
 
   public static final String TOPLOC = "TOPLOC";
@@ -167,7 +169,10 @@ public class LocationInference {
 
     this.mapDescToLocationSummary = new HashMap<Descriptor, LocationSummary>();
 
-    mapMethodDescriptorToSubGlobalFlowGraph = new HashMap<MethodDescriptor, GlobalFlowGraph>();
+    this.mapMethodDescriptorToSubGlobalFlowGraph = new HashMap<MethodDescriptor, GlobalFlowGraph>();
+
+    this.mapMethodInvokeNodeToMapCallerArgToCalleeArg =
+        new HashMap<MethodInvokeNode, Map<NTuple<Descriptor>, NTuple<Descriptor>>>();
 
   }
 
@@ -219,8 +224,6 @@ public class LocationInference {
 
     // constructGlobalFlowGraph();
 
-    System.exit(0);
-
     constructHierarchyGraph();
 
     debug_writeHierarchyDotFiles();
@@ -263,13 +266,256 @@ public class LocationInference {
 
   }
 
+  public Map<NTuple<Descriptor>, NTuple<Descriptor>> getMapCallerArgToCalleeParam(
+      MethodInvokeNode min) {
+
+    if (!mapMethodInvokeNodeToMapCallerArgToCalleeArg.containsKey(min)) {
+      mapMethodInvokeNodeToMapCallerArgToCalleeArg.put(min,
+          new HashMap<NTuple<Descriptor>, NTuple<Descriptor>>());
+    }
+
+    return mapMethodInvokeNodeToMapCallerArgToCalleeArg.get(min);
+  }
+
+  public void addMapCallerArgToCalleeParam(MethodInvokeNode min, NTuple<Descriptor> callerArg,
+      NTuple<Descriptor> calleeParam) {
+    getMapCallerArgToCalleeParam(min).put(callerArg, calleeParam);
+  }
+
   private void assignCompositeLocation() {
     calculateGlobalValueFlowCompositeLocation();
     translateCompositeLocationAssignmentToFlowGraph();
+    _debug_printGraph();
   }
 
   private void translateCompositeLocationAssignmentToFlowGraph() {
-    
+
+    MethodDescriptor methodEventLoopDesc = ssjava.getMethodContainingSSJavaLoop();
+    translateCompositeLocationAssignmentToFlowGraph(methodEventLoopDesc);
+
+  }
+
+  private void translateCompositeLocationAssignmentToFlowGraph(MethodDescriptor mdCaller) {
+
+    System.out.println("\n#translateCompositeLocationAssignmentToFlowGraph=" + mdCaller);
+
+    // First, assign a composite location to a node in the flow graph
+    GlobalFlowGraph callerGlobalFlowGraph = getSubGlobalFlowGraph(mdCaller);
+
+    FlowGraph callerFlowGraph = getFlowGraph(mdCaller);
+    Map<Location, CompositeLocation> callerMapLocToCompLoc =
+        callerGlobalFlowGraph.getMapLocationToInferCompositeLocation();
+    System.out.println("---callerMapLocToCompLoc=" + callerMapLocToCompLoc);
+    Set<Location> methodLocSet = callerMapLocToCompLoc.keySet();
+    for (Iterator iterator = methodLocSet.iterator(); iterator.hasNext();) {
+      Location methodLoc = (Location) iterator.next();
+      if (methodLoc.getDescriptor().equals(mdCaller)) {
+        CompositeLocation inferCompLoc = callerMapLocToCompLoc.get(methodLoc);
+        assignCompositeLocationToFlowGraph(callerFlowGraph, methodLoc, inferCompLoc);
+      }
+    }
+
+    Set<MethodInvokeNode> minSet = mapMethodDescriptorToMethodInvokeNodeSet.get(mdCaller);
+
+    Set<MethodDescriptor> calleeSet = new HashSet<MethodDescriptor>();
+    for (Iterator iterator = minSet.iterator(); iterator.hasNext();) {
+      MethodInvokeNode min = (MethodInvokeNode) iterator.next();
+      // need to translate a composite location that is started with the base
+      // tuple of 'min'.
+      if (mapMethodInvokeNodeToBaseTuple.get(min) != null) {
+        // if mapMethodInvokeNodeToBaseTuple doesn't have a mapping
+        // it means that the corresponding callee method does not cause any
+        // flows
+        translateMapLocationToInferCompositeLocationToCalleeGraph(callerGlobalFlowGraph, min);
+      }
+      calleeSet.add(min.getMethod());
+    }
+
+    for (Iterator iterator = calleeSet.iterator(); iterator.hasNext();) {
+      MethodDescriptor callee = (MethodDescriptor) iterator.next();
+      translateCompositeLocationAssignmentToFlowGraph(callee);
+    }
+
+  }
+
+  public void assignCompositeLocationToFlowGraph(FlowGraph flowGraph, Location loc,
+      CompositeLocation inferCompLoc) {
+    Descriptor localDesc = loc.getLocDescriptor();
+
+    Set<FlowNode> nodeSet = flowGraph.getNodeSet();
+    for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
+      FlowNode node = (FlowNode) iterator.next();
+      if (node.getDescTuple().startsWith(localDesc)) {
+        // need to assign the inferred composite location to this node
+        CompositeLocation newCompLoc = generateCompositeLocation(node.getDescTuple(), inferCompLoc);
+        node.setCompositeLocation(newCompLoc);
+        System.out.println("SET Node=" + node + "  inferCompLoc=" + newCompLoc);
+      }
+    }
+  }
+
+  private CompositeLocation generateCompositeLocation(NTuple<Descriptor> nodeDescTuple,
+      CompositeLocation inferCompLoc) {
+
+    System.out.println("generateCompositeLocation=" + nodeDescTuple + " with inferCompLoc="
+        + inferCompLoc);
+
+    CompositeLocation newCompLoc = new CompositeLocation();
+    for (int i = 0; i < inferCompLoc.getSize(); i++) {
+      newCompLoc.addLocation(inferCompLoc.get(i));
+    }
+
+    Descriptor lastDescOfPrefix = nodeDescTuple.get(0);
+    Descriptor enclosingDescriptor;
+    if (lastDescOfPrefix instanceof InterDescriptor) {
+      enclosingDescriptor = null;
+    } else {
+      enclosingDescriptor = ((VarDescriptor) lastDescOfPrefix).getType().getClassDesc();
+    }
+
+    for (int i = 1; i < nodeDescTuple.size(); i++) {
+      Descriptor desc = nodeDescTuple.get(i);
+      Location locElement = new Location(enclosingDescriptor, desc);
+      newCompLoc.addLocation(locElement);
+
+      enclosingDescriptor = ((FieldDescriptor) desc).getClassDescriptor();
+    }
+
+    return newCompLoc;
+  }
+
+  private void translateMapLocationToInferCompositeLocationToCalleeGraph(
+      GlobalFlowGraph callerGraph, MethodInvokeNode min) {
+
+    MethodDescriptor mdCallee = min.getMethod();
+    MethodDescriptor mdCaller = callerGraph.getMethodDescriptor();
+    Map<Location, CompositeLocation> callerMapLocToCompLoc =
+        callerGraph.getMapLocationToInferCompositeLocation();
+
+    FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
+    GlobalFlowGraph calleeGlobalGraph = getSubGlobalFlowGraph(mdCallee);
+
+    NTuple<Location> baseLocTuple =
+        translateToLocTuple(mdCaller, mapMethodInvokeNodeToBaseTuple.get(min));
+
+    System.out.println("\n-translate caller infer composite loc to callee=" + mdCallee
+        + " baseLocTuple=" + baseLocTuple);
+    Set<Location> keySet = callerMapLocToCompLoc.keySet();
+    for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
+      Location key = (Location) iterator.next();
+      CompositeLocation callerCompLoc = callerMapLocToCompLoc.get(key);
+
+      if (!key.getDescriptor().equals(mdCaller)) {
+        System.out.println("--- caller key=" + key + "  callerCompLoc=" + callerCompLoc);
+
+        // && callerCompLoc.getTuple().startsWith(baseLocTuple)) {
+        // need to translate to the callee side
+
+        // TODO
+        CompositeLocation newCalleeCompLoc;
+        if (callerCompLoc.getTuple().startsWith(baseLocTuple)) {
+          System.out.println("---need to translate callerCompLoc=" + callerCompLoc
+              + " with baseTuple=" + baseLocTuple);
+          newCalleeCompLoc =
+              translateCompositeLocationToCallee(callerCompLoc, baseLocTuple, mdCallee);
+
+          calleeGlobalGraph.addMapLocationToInferCompositeLocation(key, newCalleeCompLoc);
+          System.out.println("---callee loc=" + key + "  newCalleeCompLoc=" + newCalleeCompLoc);
+        } else {
+          // newCalleeCompLoc = callerCompLoc.clone();
+        }
+
+      }
+    }
+
+    // If the location of an argument has a composite location
+    // need to assign a proper composite location to the corresponding callee parameter
+    System.out.println("\n-translate arg composite location to callee param. min="
+        + min.printNode(0));
+    Map<Integer, NTuple<Descriptor>> mapIdxToArgTuple = mapMethodInvokeNodeToArgIdxMap.get(min);
+    Set<Integer> idxSet = mapIdxToArgTuple.keySet();
+    for (Iterator iterator = idxSet.iterator(); iterator.hasNext();) {
+      Integer idx = (Integer) iterator.next();
+
+      if (idx == 0 && !min.getMethod().isStatic()) {
+        continue;
+      }
+
+      NTuple<Descriptor> argTuple = mapIdxToArgTuple.get(idx);
+      if (argTuple.size() > 0) {
+        // check if an arg tuple has been already assigned to a composite location
+        NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argTuple);
+        Location argLocalLoc = argLocTuple.get(0);
+
+        // if (!isPrimitiveType(argTuple)) {
+        if (callerMapLocToCompLoc.containsKey(argLocalLoc)) {
+
+          CompositeLocation callerCompLoc = callerMapLocToCompLoc.get(argLocalLoc);
+          for (int i = 1; i < argLocTuple.size(); i++) {
+            callerCompLoc.addLocation(argLocTuple.get(i));
+          }
+
+          if (callerCompLoc.getTuple().startsWith(baseLocTuple)) {
+
+            FlowNode calleeParamFlowNode = calleeFlowGraph.getParamFlowNode(idx);
+            NTuple<Descriptor> calleeParamDescTuple = calleeParamFlowNode.getDescTuple();
+            NTuple<Location> calleeParamLocTuple =
+                translateToLocTuple(mdCallee, calleeParamDescTuple);
+
+            System.out.println("---need to translate callerCompLoc=" + callerCompLoc
+                + " with baseTuple=" + baseLocTuple + "   calleeParamLocTuple="
+                + calleeParamLocTuple);
+
+            CompositeLocation newCalleeCompLoc =
+                translateCompositeLocationToCallee(callerCompLoc, baseLocTuple, mdCallee);
+
+            calleeGlobalGraph.addMapLocationToInferCompositeLocation(calleeParamLocTuple.get(0),
+                newCalleeCompLoc);
+
+            System.out.println("---callee loc=" + calleeParamLocTuple.get(0)
+                + "  newCalleeCompLoc=" + newCalleeCompLoc);
+
+            // System.out.println("###need to assign composite location to=" + calleeParamDescTuple
+            // + " with baseTuple=" + baseLocTuple);
+          }
+
+        }
+      }
+
+    }
+
+  }
+
+  private boolean isPrimitiveType(NTuple<Descriptor> argTuple) {
+
+    Descriptor lastDesc = argTuple.get(argTuple.size() - 1);
+
+    if (lastDesc instanceof FieldDescriptor) {
+      return ((FieldDescriptor) lastDesc).getType().isPrimitive();
+    } else if (lastDesc instanceof VarDescriptor) {
+      return ((VarDescriptor) lastDesc).getType().isPrimitive();
+    }
+
+    return true;
+  }
+
+  private CompositeLocation translateCompositeLocationToCallee(CompositeLocation callerCompLoc,
+      NTuple<Location> baseLocTuple, MethodDescriptor mdCallee) {
+
+    CompositeLocation newCalleeCompLoc = new CompositeLocation();
+
+    Location calleeThisLoc = new Location(mdCallee, mdCallee.getThis());
+    newCalleeCompLoc.addLocation(calleeThisLoc);
+
+    // remove the base tuple from the caller
+    // ex; In the method invoation foo.bar.methodA(), the callee will have the composite location
+    // ,which is relative to the 'this' variable, <THIS,...>
+    for (int i = baseLocTuple.size(); i < callerCompLoc.getSize(); i++) {
+      newCalleeCompLoc.addLocation(callerCompLoc.get(i));
+    }
+
+    return newCalleeCompLoc;
+
   }
 
   private void constructGlobalFlowGraph() {
@@ -289,7 +535,7 @@ public class LocationInference {
 
         FlowGraph flowGraph = getFlowGraph(md);
         FlowGraph subGlobalFlowGraph = flowGraph.clone();
-        
+
         // mapMethodDescriptorToSubGlobalFlowGraph.put(md, subGlobalFlowGraph);
 
         // addValueFlowsFromCalleeSubGlobalFlowGraph(md, subGlobalFlowGraph);
@@ -308,7 +554,8 @@ public class LocationInference {
 
     // DEBUG: write a global flow graph
     MethodDescriptor mdContainingSSJavaLoop = ssjava.getMethodContainingSSJavaLoop();
-    // FlowGraph globalFlowGraph = getSubGlobalFlowGraph(mdContainingSSJavaLoop);
+    // FlowGraph globalFlowGraph =
+    // getSubGlobalFlowGraph(mdContainingSSJavaLoop);
     // System.out.println("GLOBAL NODE SET=" + globalFlowGraph.getNodeSet());
     // assignCompositeLocation(globalFlowGraph);
     // try {
@@ -324,28 +571,39 @@ public class LocationInference {
 
     System.out.println("SSJAVA: Calculate composite locations in the global value flow graph");
     MethodDescriptor methodDescEventLoop = ssjava.getMethodContainingSSJavaLoop();
-    GlobalFlowGraph graph = getSubGlobalFlowGraph(methodDescEventLoop);
+    GlobalFlowGraph globalFlowGraph = getSubGlobalFlowGraph(methodDescEventLoop);
 
-    Set<NTuple<Location>> prefixSet = new HashSet<NTuple<Location>>();
+    Set<Location> calculatedPrefixSet = new HashSet<Location>();
 
-    Set<GlobalFlowNode> nodeSet = graph.getNodeSet();
+    Set<GlobalFlowNode> nodeSet = globalFlowGraph.getNodeSet();
 
     next: for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
       GlobalFlowNode node = (GlobalFlowNode) iterator.next();
-      Set<GlobalFlowNode> incomingNodeSet = graph.getIncomingNodeSet(node);
 
-      List<NTuple<Location>> prefixList = calculatePrefixList(graph, node);
+      Location prefixLoc = node.getLocTuple().get(0);
+
+      if (calculatedPrefixSet.contains(prefixLoc)) {
+        // the prefix loc has been already assigned to a composite location
+        continue;
+      }
+
+      calculatedPrefixSet.add(prefixLoc);
 
-      Set<GlobalFlowNode> reachNodeSet = graph.getReachableNodeSetFrom(node);
+      // Set<GlobalFlowNode> incomingNodeSet = globalFlowGraph.getIncomingNodeSet(node);
+      List<NTuple<Location>> prefixList = calculatePrefixList(globalFlowGraph, node);
 
-      // System.out.println("node=" + node + "    inNodeSet=" + incomingNodeSet
-      // + "   reachableNodeSet=" + reachNodeSet);
+      Set<GlobalFlowNode> reachableNodeSet =
+          globalFlowGraph.getReachableNodeSetByPrefix(node.getLocTuple().get(0));
+      // Set<GlobalFlowNode> reachNodeSet = globalFlowGraph.getReachableNodeSetFrom(node);
+
+      // System.out.println("node=" + node + "    prefixList=" + prefixList + "   reachableNodeSet="
+      // + reachableNodeSet);
 
       for (int i = 0; i < prefixList.size(); i++) {
         NTuple<Location> curPrefix = prefixList.get(i);
         Set<NTuple<Location>> reachableCommonPrefixSet = new HashSet<NTuple<Location>>();
 
-        for (Iterator iterator2 = reachNodeSet.iterator(); iterator2.hasNext();) {
+        for (Iterator iterator2 = reachableNodeSet.iterator(); iterator2.hasNext();) {
           GlobalFlowNode reachNode = (GlobalFlowNode) iterator2.next();
           if (reachNode.getLocTuple().startsWith(curPrefix)) {
             reachableCommonPrefixSet.add(reachNode.getLocTuple());
@@ -353,29 +611,83 @@ public class LocationInference {
         }
 
         if (!reachableCommonPrefixSet.isEmpty()) {
-          CompositeLocation newCompLoc = generateCompositeLocation(curPrefix);
-          System.out.println("NEED TO ASSIGN COMP LOC TO " + node + " with prefix=" + curPrefix);
-          System.out.println("- newCompLoc=" + newCompLoc);
-          // flowNode.setCompositeLocation(newCompLoc);
-          continue next;
+
+          MethodDescriptor curPrefixFirstElementMethodDesc =
+              (MethodDescriptor) curPrefix.get(0).getDescriptor();
+
+          MethodDescriptor nodePrefixLocFirstElementMethodDesc =
+              (MethodDescriptor) prefixLoc.getDescriptor();
+
+          if (curPrefixFirstElementMethodDesc.equals(nodePrefixLocFirstElementMethodDesc)
+              || isTransitivelyCalledFrom(nodePrefixLocFirstElementMethodDesc,
+                  curPrefixFirstElementMethodDesc)) {
+
+            // TODO
+            // if (!node.getLocTuple().startsWith(curPrefix.get(0))) {
+
+            Location curPrefixLocalLoc = curPrefix.get(0);
+            if (globalFlowGraph.mapLocationToInferCompositeLocation.containsKey(curPrefixLocalLoc)) {
+              // in this case, the local variable of the current prefix has already got a composite
+              // location
+              // so we just ignore the current composite location.
+
+              // System.out.println("HERE WE DO NOT ASSIGN A COMPOSITE LOCATION TO =" + node
+              // + " DUE TO " + curPrefix);
+
+              continue next;
+            }
+
+            Location targetLocalLoc = node.getLocTuple().get(0);
+            // CompositeLocation curCompLoc = globalFlowGraph.getCompositeLocation(targetLocalLoc);
+            // if ((curPrefix.size() + 1) > curCompLoc.getSize()) {
+
+            CompositeLocation newCompLoc = generateCompositeLocation(curPrefix);
+            System.out.println("NEED TO ASSIGN COMP LOC TO " + node + " with prefix=" + curPrefix);
+            System.out.println("- newCompLoc=" + newCompLoc);
+            globalFlowGraph.addMapLocationToInferCompositeLocation(targetLocalLoc, newCompLoc);
+            // }
+
+            continue next;
+            // }
+
+          }
+
         }
 
       }
 
     }
-    // Set<GlobalFlowNode> inNodeSet = graph.getIncomingNodeSetWithPrefix(prefix);
+    // Set<GlobalFlowNode> inNodeSet =
+    // graph.getIncomingNodeSetWithPrefix(prefix);
     // System.out.println("inNodeSet=" + inNodeSet + "  from=" + node);
   }
 
+  private void assignCompositeLocation(CompositeLocation compLocPrefix, GlobalFlowNode node) {
+    CompositeLocation newCompLoc = compLocPrefix.clone();
+    NTuple<Location> locTuple = node.getLocTuple();
+    for (int i = 1; i < locTuple.size(); i++) {
+      newCompLoc.addLocation(locTuple.get(i));
+    }
+    node.setInferCompositeLocation(newCompLoc);
+  }
+
   private List<NTuple<Location>> calculatePrefixList(GlobalFlowGraph graph, GlobalFlowNode node) {
 
-    System.out.println("\n##### calculatePrefixList=" + node);
+    System.out.println("\n##### calculatePrefixList node=" + node);
+
+    MethodDescriptor md = graph.getMethodDescriptor();
 
-    Set<GlobalFlowNode> incomingNodeSet = graph.getIncomingNodeSet(node);
+    Set<GlobalFlowNode> incomingNodeSetPrefix =
+        graph.getIncomingNodeSetByPrefix(node.getLocTuple().get(0));
+    // System.out.println("incomingNodeSetPrefix=" + incomingNodeSetPrefix);
+    //
+    // Set<GlobalFlowNode> reachableNodeSetPrefix =
+    // graph.getReachableNodeSetByPrefix(node.getLocTuple().get(0));
+    // System.out.println("reachableNodeSetPrefix=" + reachableNodeSetPrefix);
 
     List<NTuple<Location>> prefixList = new ArrayList<NTuple<Location>>();
 
-    for (Iterator iterator = incomingNodeSet.iterator(); iterator.hasNext();) {
+    for (Iterator iterator = incomingNodeSetPrefix.iterator(); iterator.hasNext();) {
       GlobalFlowNode inNode = (GlobalFlowNode) iterator.next();
       NTuple<Location> inNodeTuple = inNode.getLocTuple();
 
@@ -401,6 +713,35 @@ public class LocationInference {
       }
     });
     return prefixList;
+
+    // List<NTuple<Location>> prefixList = new ArrayList<NTuple<Location>>();
+    //
+    // for (Iterator iterator = incomingNodeSet.iterator(); iterator.hasNext();) {
+    // GlobalFlowNode inNode = (GlobalFlowNode) iterator.next();
+    // NTuple<Location> inNodeTuple = inNode.getLocTuple();
+    //
+    // for (int i = 1; i < inNodeTuple.size(); i++) {
+    // NTuple<Location> prefix = inNodeTuple.subList(0, i);
+    // if (!prefixList.contains(prefix)) {
+    // prefixList.add(prefix);
+    // }
+    // }
+    // }
+    //
+    // Collections.sort(prefixList, new Comparator<NTuple<Location>>() {
+    // public int compare(NTuple<Location> arg0, NTuple<Location> arg1) {
+    // int s0 = arg0.size();
+    // int s1 = arg1.size();
+    // if (s0 > s1) {
+    // return -1;
+    // } else if (s0 == s1) {
+    // return 0;
+    // } else {
+    // return 1;
+    // }
+    // }
+    // });
+    // return prefixList;
   }
 
   private GlobalFlowGraph constructSubGlobalFlowGraph(FlowGraph flowGraph) {
@@ -418,11 +759,13 @@ public class LocationInference {
       NTuple<Descriptor> srcDescTuple = edge.getInitTuple();
       NTuple<Descriptor> dstDescTuple = edge.getEndTuple();
 
-      // here only keep the first element(method location) of the descriptor tuple
+      // here only keep the first element(method location) of the descriptor
+      // tuple
       NTuple<Location> srcLocTuple = translateToLocTuple(md, srcDescTuple);
       // Location srcMethodLoc = srcLocTuple.get(0);
       // Descriptor srcVarDesc = srcMethodLoc.getLocDescriptor();
-      // // if (flowGraph.isParamDesc(srcVarDesc) && (!srcVarDesc.equals(md.getThis()))) {
+      // // if (flowGraph.isParamDesc(srcVarDesc) &&
+      // (!srcVarDesc.equals(md.getThis()))) {
       // if (!srcVarDesc.equals(md.getThis())) {
       // srcLocTuple = new NTuple<Location>();
       // Location loc = new Location(md, srcVarDesc);
@@ -450,6 +793,7 @@ public class LocationInference {
     NTuple<Location> locTuple = new NTuple<Location>();
 
     Descriptor enclosingDesc = md;
+    // System.out.println("md=" + md + "  descTuple=" + descTuple);
     for (int i = 0; i < descTuple.size(); i++) {
       Descriptor desc = descTuple.get(i);
 
@@ -505,16 +849,30 @@ public class LocationInference {
   private void propagateValueFlowsToCallerFromSubGlobalFlowGraph(MethodInvokeNode min,
       MethodDescriptor mdCaller, MethodDescriptor possibleMdCallee) {
 
-    NTuple<Descriptor> baseTuple = mapMethodInvokeNodeToBaseTuple.get(min);
+    System.out.println("propagateValueFlowsToCallerFromSubGlobalFlowGraph=" + min.printNode(0)
+        + " by caller=" + mdCaller);
+    FlowGraph calleeFlowGraph = getFlowGraph(possibleMdCallee);
+    Map<Integer, NTuple<Descriptor>> mapIdxToArg = mapMethodInvokeNodeToArgIdxMap.get(min);
 
-    NTuple<Location> baseLocTuple = translateToLocTuple(mdCaller, baseTuple);
+    System.out.println("mapMethodInvokeNodeToArgIdxMap.get(min)="
+        + mapMethodInvokeNodeToArgIdxMap.get(min));
+    Set<Integer> keySet = mapIdxToArg.keySet();
+    for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
+      Integer idx = (Integer) iterator.next();
+      NTuple<Descriptor> argDescTuple = mapIdxToArg.get(idx);
+      if (argDescTuple.size() > 0) {
+        NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argDescTuple);
+        NTuple<Descriptor> paramDescTuple = calleeFlowGraph.getParamFlowNode(idx).getDescTuple();
+        NTuple<Location> paramLocTuple = translateToLocTuple(possibleMdCallee, paramDescTuple);
+        addMapCallerArgToCalleeParam(min, argDescTuple, paramDescTuple);
+      }
 
-    FlowGraph calleeFlowGraph = getFlowGraph(possibleMdCallee);
+    }
 
-    GlobalFlowGraph callerSubGlobalGraph = getSubGlobalFlowGraph(mdCaller);
+    NTuple<Descriptor> baseTuple = mapMethodInvokeNodeToBaseTuple.get(min);
     GlobalFlowGraph calleeSubGlobalGraph = getSubGlobalFlowGraph(possibleMdCallee);
-
     Set<GlobalFlowNode> calleeNodeSet = calleeSubGlobalGraph.getNodeSet();
+    System.out.println("#calleeNodeSet=" + calleeNodeSet);
     for (Iterator iterator = calleeNodeSet.iterator(); iterator.hasNext();) {
       GlobalFlowNode calleeNode = (GlobalFlowNode) iterator.next();
       addValueFlowFromCalleeNode(min, mdCaller, possibleMdCallee, calleeNode);
@@ -528,30 +886,40 @@ public class LocationInference {
     // NTuple<Location> paramLocTuple =
     // translateToLocTuple(possibleMdCallee, paramNode.getCurrentDescTuple());
     //
-    // GlobalFlowNode globalParamNode = calleeSubGlobalGraph.getFlowNode(paramLocTuple);
+    // GlobalFlowNode globalParamNode =
+    // calleeSubGlobalGraph.getFlowNode(paramLocTuple);
     //
-    // NTuple<Descriptor> argTuple = mapMethodInvokeNodeToArgIdxMap.get(min).get(idx);
+    // NTuple<Descriptor> argTuple =
+    // mapMethodInvokeNodeToArgIdxMap.get(min).get(idx);
     //
     // NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argTuple);
     //
-    // System.out.println("argTupleSet=" + argLocTuple + "   param=" + paramLocTuple);
-    // // here, it adds all value flows reachable from the paramNode in the callee's flow graph
+    // System.out.println("argTupleSet=" + argLocTuple + "   param=" +
+    // paramLocTuple);
+    // // here, it adds all value flows reachable from the paramNode in the
+    // callee's flow graph
     //
-    // addValueFlowsFromCalleeParam(mdCaller, argLocTuple, baseLocTuple, possibleMdCallee,
+    // addValueFlowsFromCalleeParam(mdCaller, argLocTuple, baseLocTuple,
+    // possibleMdCallee,
     // globalParamNode);
     // }
     //
     // // TODO
     // // FlowGraph callerSubGlobalGraph = getSubGlobalFlowGraph(mdCaller);
-    // // FlowGraph calleeSubGlobalGraph = getSubGlobalFlowGraph(possibleMdCallee);
+    // // FlowGraph calleeSubGlobalGraph =
+    // getSubGlobalFlowGraph(possibleMdCallee);
     // //
     // // int numParam = calleeSubGlobalGraph.getNumParameters();
     // // for (int idx = 0; idx < numParam; idx++) {
     // // FlowNode paramNode = calleeSubGlobalGraph.getParamFlowNode(idx);
-    // // NTuple<Descriptor> argTuple = mapMethodInvokeNodeToArgIdxMap.get(min).get(idx);
-    // // System.out.println("argTupleSet=" + argTuple + "   param=" + paramNode);
-    // // // here, it adds all value flows reachable from the paramNode in the callee's flow graph
-    // // addValueFlowsFromCalleeParam(min, calleeSubGlobalGraph, paramNode, callerSubGlobalGraph,
+    // // NTuple<Descriptor> argTuple =
+    // mapMethodInvokeNodeToArgIdxMap.get(min).get(idx);
+    // // System.out.println("argTupleSet=" + argTuple + "   param=" +
+    // paramNode);
+    // // // here, it adds all value flows reachable from the paramNode in the
+    // callee's flow graph
+    // // addValueFlowsFromCalleeParam(min, calleeSubGlobalGraph, paramNode,
+    // callerSubGlobalGraph,
     // // argTuple, baseTuple);
     // // }
 
@@ -563,30 +931,47 @@ public class LocationInference {
     GlobalFlowGraph calleeSubGlobalGraph = getSubGlobalFlowGraph(mdCallee);
     GlobalFlowGraph callerSubGlobalGraph = getSubGlobalFlowGraph(mdCaller);
 
-    NTuple<Location> srcNodeLocTuple =
+    NTuple<Location> callerSrcNodeLocTuple =
         translateToCallerLocTuple(min, mdCallee, mdCaller, calleeSrcNode.getLocTuple());
 
-    Set<GlobalFlowNode> outNodeSet = calleeSubGlobalGraph.getOutNodeSet(calleeSrcNode);
+    if (callerSrcNodeLocTuple != null) {
+      Set<GlobalFlowNode> outNodeSet = calleeSubGlobalGraph.getOutNodeSet(calleeSrcNode);
 
-    for (Iterator iterator = outNodeSet.iterator(); iterator.hasNext();) {
-      GlobalFlowNode outNode = (GlobalFlowNode) iterator.next();
-      NTuple<Location> dstNodeLocTuple =
-          translateToCallerLocTuple(min, mdCallee, mdCaller, outNode.getLocTuple());
-      callerSubGlobalGraph.addValueFlowEdge(srcNodeLocTuple, dstNodeLocTuple);
+      for (Iterator iterator = outNodeSet.iterator(); iterator.hasNext();) {
+        GlobalFlowNode outNode = (GlobalFlowNode) iterator.next();
+        NTuple<Location> callerDstNodeLocTuple =
+            translateToCallerLocTuple(min, mdCallee, mdCaller, outNode.getLocTuple());
+        if (callerDstNodeLocTuple != null) {
+          callerSubGlobalGraph.addValueFlowEdge(callerSrcNodeLocTuple, callerDstNodeLocTuple);
+        }
+      }
     }
 
   }
 
   private NTuple<Location> translateToCallerLocTuple(MethodInvokeNode min,
       MethodDescriptor mdCallee, MethodDescriptor mdCaller, NTuple<Location> nodeLocTuple) {
+    // this method will return NULL if the corresponding argument is literal
+    // value.
+    // assumes that we don't need to propagate callee flows to the argument
+    // which is literal.
+
+    System.out.println("---translateToCallerLocTuple=" + min.printNode(0)
+        + "  callee nodeLocTuple=" + nodeLocTuple);
 
     FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
 
     NTuple<Descriptor> nodeDescTuple = translateToDescTuple(nodeLocTuple);
-
     if (calleeFlowGraph.isParameter(nodeDescTuple)) {
       int paramIdx = calleeFlowGraph.getParamIdx(nodeDescTuple);
       NTuple<Descriptor> argDescTuple = mapMethodInvokeNodeToArgIdxMap.get(min).get(paramIdx);
+      // System.out.println(" mapMethodInvokeNodeToArgIdxMap.get(min)="
+      // + mapMethodInvokeNodeToArgIdxMap.get(min));
+
+      if (argDescTuple.size() == 0) {
+        // argument is literal
+        return null;
+      }
       NTuple<Location> argLocTuple = translateToLocTuple(mdCaller, argDescTuple);
 
       NTuple<Location> callerLocTuple = new NTuple<Location>();
@@ -597,7 +982,7 @@ public class LocationInference {
       }
       return callerLocTuple;
     } else {
-      return nodeLocTuple;
+      return nodeLocTuple.clone();
     }
 
   }
@@ -636,7 +1021,8 @@ public class LocationInference {
     // curNodeLocTuple = translateToCaller(argLocTuple, curNodeLocTuple);
     // }
     //
-    // Set<GlobalFlowNode> outNodeSet = calleeSubGlobalGraph.getOutNodeSet(calleeCurNode);
+    // Set<GlobalFlowNode> outNodeSet =
+    // calleeSubGlobalGraph.getOutNodeSet(calleeCurNode);
     // for (Iterator iterator = outNodeSet.iterator(); iterator.hasNext();) {
     // GlobalFlowNode outNode = (GlobalFlowNode) iterator.next();
     //
@@ -654,12 +1040,14 @@ public class LocationInference {
     // // destination node is started with 'parameter'
     // // need to translate it in terms of the caller's a node
     // srcDescTuple =
-    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(srcDescTuple), srcDescTuple);
+    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(srcDescTuple),
+    // srcDescTuple);
     // }
     //
     // }
     //
-    // Set<FlowEdge> edgeSet = calleeSubGlobalGraph.getOutEdgeSetStartingFrom(calleeSrcNode);
+    // Set<FlowEdge> edgeSet =
+    // calleeSubGlobalGraph.getOutEdgeSetStartingFrom(calleeSrcNode);
     // for (Iterator iterator = edgeSet.iterator(); iterator.hasNext();) {
     // FlowEdge flowEdge = (FlowEdge) iterator.next();
     //
@@ -672,21 +1060,24 @@ public class LocationInference {
     // // destination node is started with 'parameter'
     // // need to translate it in terms of the caller's a node
     // srcDescTuple =
-    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(srcDescTuple), srcDescTuple);
+    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(srcDescTuple),
+    // srcDescTuple);
     // }
     //
     // if (calleeSubGlobalGraph.isParameter(dstDescTuple)) {
     // // destination node is started with 'parameter'
     // // need to translate it in terms of the caller's a node
     // dstDescTuple =
-    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(dstDescTuple), dstDescTuple);
+    // translateToCaller(min, calleeSubGlobalGraph.getParamIdx(dstDescTuple),
+    // dstDescTuple);
     // }
     //
     // callerSubGlobalGraph.addValueFlowEdge(srcDescTuple, dstDescTuple);
     //
     // if (!visited.contains(dstNode)) {
     // visited.add(dstNode);
-    // recurAddValueFlowsFromCalleeParam(min, calleeSubGlobalGraph, dstNode, callerSubGlobalGraph,
+    // recurAddValueFlowsFromCalleeParam(min, calleeSubGlobalGraph, dstNode,
+    // callerSubGlobalGraph,
     // dstDescTuple, visited, baseTuple);
     // }
     //
@@ -713,7 +1104,8 @@ public class LocationInference {
 
     MethodDescriptor mdCallee = calleeSubGlobalGraph.getMethodDescriptor();
 
-    // Set<FlowEdge> edgeSet = calleeSubGlobalGraph.getOutEdgeSet(calleeSrcNode);
+    // Set<FlowEdge> edgeSet =
+    // calleeSubGlobalGraph.getOutEdgeSet(calleeSrcNode);
     Set<FlowEdge> edgeSet = calleeSubGlobalGraph.getOutEdgeSetStartingFrom(calleeSrcNode);
     for (Iterator iterator = edgeSet.iterator(); iterator.hasNext();) {
       FlowEdge flowEdge = (FlowEdge) iterator.next();
@@ -805,7 +1197,8 @@ public class LocationInference {
       FlowGraph flowGraph = getFlowGraph(md);
       MethodSummary methodSummary = getMethodSummary(md);
 
-      // construct a parameter mapping that maps a parameter descriptor to an inferred composite
+      // construct a parameter mapping that maps a parameter descriptor to an
+      // inferred composite
       // location
 
       for (int paramIdx = 0; paramIdx < flowGraph.getNumParameters(); paramIdx++) {
@@ -925,14 +1318,16 @@ public class LocationInference {
 
       // System.out.println("\nSSJAVA: Insering Combination Nodes:" + desc);
       // HierarchyGraph skeletonGraph = getSkeletonHierarchyGraph(desc);
-      // HierarchyGraph skeletonGraphWithCombinationNode = skeletonGraph.clone();
+      // HierarchyGraph skeletonGraphWithCombinationNode =
+      // skeletonGraph.clone();
       // skeletonGraphWithCombinationNode.setName(desc + "_SC");
       //
       // HierarchyGraph simpleHierarchyGraph = getSimpleHierarchyGraph(desc);
       // System.out.println("Identifying Combination Nodes:");
       // skeletonGraphWithCombinationNode.insertCombinationNodesToGraph(simpleHierarchyGraph);
       // skeletonGraphWithCombinationNode.simplifySkeletonCombinationHierarchyGraph();
-      // mapDescriptorToCombineSkeletonHierarchyGraph.put(desc, skeletonGraphWithCombinationNode);
+      // mapDescriptorToCombineSkeletonHierarchyGraph.put(desc,
+      // skeletonGraphWithCombinationNode);
     }
 
   }
@@ -1870,7 +2265,8 @@ public class LocationInference {
           String paramLocLocalSymbol = inferLoc.get(0).getLocIdentifier();
           if (!methodLattice.isGreaterThan(paramLocLocalSymbol, returnLocSymbol)) {
             // TODO
-            // addRelationHigherToLower(methodLattice, methodInfo, paramLocLocalSymbol,
+            // addRelationHigherToLower(methodLattice, methodInfo,
+            // paramLocLocalSymbol,
             // returnLocSymbol);
           }
         }
@@ -1881,7 +2277,8 @@ public class LocationInference {
               generateInferredCompositeLocation(methodInfo, fg.getLocationTuple(returnNode));
           if (!isGreaterThan(methodLattice, inferLoc, returnLocInferLoc)) {
             // TODO
-            // addRelation(methodLattice, methodInfo, inferLoc, returnLocInferLoc);
+            // addRelation(methodLattice, methodInfo, inferLoc,
+            // returnLocInferLoc);
           }
         }
 
@@ -2045,12 +2442,13 @@ public class LocationInference {
           // parameters
 
           Set<FlowNode> localReachSet = calleeFlowGraph.getLocalReachFlowNodeSetFrom(paramNode1);
+          System.out.println("-param1=" + paramNode1 + " is higher than param2=" + paramNode2);
+          System.out.println("-- localReachSet from param1=" + localReachSet);
 
-          if (localReachSet.contains(paramNode2)) {
+          if (arg1Tuple.size() > 0 && arg2Tuple.size() > 0 && localReachSet.contains(paramNode2)) {
             // need to propagate an ordering relation s.t. arg1 is higher
             // than arg2
 
-            System.out.println("-param1=" + paramNode1 + " is higher than param2=" + paramNode2);
             System.out
                 .println("-arg1Tuple=" + arg1Tuple + " is higher than arg2Tuple=" + arg2Tuple);
 
@@ -2078,8 +2476,10 @@ public class LocationInference {
     // graph
 
     // TODO
-    // also if a parameter is a composite location and is started with "this" reference,
-    // need to make sure that the corresponding argument is higher than the translated location of
+    // also if a parameter is a composite location and is started with "this"
+    // reference,
+    // need to make sure that the corresponding argument is higher than the
+    // translated location of
     // the parameter.
 
     FlowGraph calleeFlowGraph = getFlowGraph(mdCallee);
@@ -2138,7 +2538,8 @@ public class LocationInference {
                   // calculate the prefix of the argument
 
                   if (arg2Tuple.size() == 1 && arg2Tuple.get(0).equals(baseRef)) {
-                    // in this case, the callee flow causes a caller flow to the object whose method
+                    // in this case, the callee flow causes a caller flow to the
+                    // object whose method
                     // is invoked.
 
                     if (!paramNode1.getCurrentDescTuple().startsWith(mdCallee.getThis())) {
@@ -2149,8 +2550,10 @@ public class LocationInference {
                               paramNode1);
 
                       if (param1Prefix != null && param1Prefix.startsWith(mdCallee.getThis())) {
-                        // in this case, we need to create a new edge 'this.FIELD'->'this'
-                        // but we couldn't... instead we assign a new composite location started
+                        // in this case, we need to create a new edge
+                        // 'this.FIELD'->'this'
+                        // but we couldn't... instead we assign a new composite
+                        // location started
                         // with 'this' reference to the corresponding parameter
 
                         CompositeLocation compLocForParam1 =
@@ -2160,8 +2563,10 @@ public class LocationInference {
                             .println("set comp loc=" + compLocForParam1 + " to " + paramNode1);
                         paramNode1.setCompositeLocation(compLocForParam1);
 
-                        // then, we need to make sure that the corresponding argument in the caller
-                        // is required to be higher than or equal to the translated parameter
+                        // then, we need to make sure that the corresponding
+                        // argument in the caller
+                        // is required to be higher than or equal to the
+                        // translated parameter
                         // location
 
                         NTuple<Descriptor> translatedParamTuple =
@@ -2196,7 +2601,8 @@ public class LocationInference {
                     }
 
                   } else if (arg1Tuple.size() == 1 && arg1Tuple.get(0).equals(baseRef)) {
-                    // in this case, the callee flow causes a caller flow originated from the object
+                    // in this case, the callee flow causes a caller flow
+                    // originated from the object
                     // whose
                     // method is invoked.
 
@@ -2210,8 +2616,10 @@ public class LocationInference {
 
                       if (param2Prefix != null && param2Prefix.startsWith(mdCallee.getThis())) {
                         // in this case, we need to create a new edge 'this' ->
-                        // 'this.FIELD' but we couldn't... instead we assign the corresponding
-                        // parameter a new composite location started with 'this' reference
+                        // 'this.FIELD' but we couldn't... instead we assign the
+                        // corresponding
+                        // parameter a new composite location started with
+                        // 'this' reference
 
                         CompositeLocation compLocForParam2 =
                             generateCompositeLocation(mdCallee, param2Prefix);
@@ -2586,11 +2994,6 @@ public class LocationInference {
     return false;
   }
 
-  public boolean isPrimitiveLocalVariable(FlowNode node) {
-    VarDescriptor varDesc = (VarDescriptor) node.getDescTuple().get(0);
-    return varDesc.getType().isPrimitive();
-  }
-
   private SSJavaLattice<String> getLattice(Descriptor d) {
     if (d instanceof MethodDescriptor) {
       return getMethodLattice((MethodDescriptor) d);
@@ -2616,6 +3019,7 @@ public class LocationInference {
     NTuple<Descriptor> srcCurTuple = srcNode.getCurrentDescTuple();
     NTuple<Descriptor> dstCurTuple = dstNode.getCurrentDescTuple();
 
+    System.out.println("srcCurTuple=" + srcCurTuple + "  dstCurTuple=" + dstCurTuple);
     if (srcCurTuple.get(idx).equals(dstCurTuple.get(idx)) && srcCurTuple.size() > (idx + 1)
         && dstCurTuple.size() > (idx + 1)) {
       // value flow between fields: we don't need to add a binary relation
@@ -2627,9 +3031,14 @@ public class LocationInference {
       if (idx == 0) {
         classDesc = ((VarDescriptor) desc).getType().getClassDesc();
       } else {
-        classDesc = ((FieldDescriptor) desc).getType().getClassDesc();
-      }
 
+        if (desc instanceof FieldDescriptor) {
+          classDesc = ((FieldDescriptor) desc).getType().getClassDesc();
+        } else {
+          // TODO: need to handle a location descriptor case
+          classDesc = null;
+        }
+      }
       extractFlowsBetweenFields(classDesc, srcNode, dstNode, idx + 1);
 
     } else {
@@ -2707,6 +3116,21 @@ public class LocationInference {
 
   }
 
+  public boolean isTransitivelyCalledFrom(MethodDescriptor callee, MethodDescriptor caller) {
+    // if the callee is transitively invoked from the caller
+    // return true;
+
+    int callerIdx = toanalyze_methodDescList.indexOf(caller);
+    int calleeIdx = toanalyze_methodDescList.indexOf(callee);
+
+    if (callerIdx < calleeIdx) {
+      return true;
+    }
+
+    return false;
+
+  }
+
   public void constructFlowGraph() {
 
     System.out.println("");
@@ -2742,21 +3166,42 @@ public class LocationInference {
 
         analyzeMethodBody(md.getClassDesc(), md);
 
+        // System.out.println("##constructSubGlobalFlowGraph");
+        // GlobalFlowGraph subGlobalFlowGraph = constructSubGlobalFlowGraph(fg);
+        // mapMethodDescriptorToSubGlobalFlowGraph.put(md, subGlobalFlowGraph);
+        //
+        // // TODO
+        // System.out.println("##addValueFlowsFromCalleeSubGlobalFlowGraph");
+        // addValueFlowsFromCalleeSubGlobalFlowGraph(md, subGlobalFlowGraph);
+        // subGlobalFlowGraph.writeGraph("_SUBGLOBAL");
+        //
+        // propagateFlowsFromCalleesWithNoCompositeLocation(md);
+
+      }
+    }
+    // _debug_printGraph();
+
+    methodDescList = (LinkedList<MethodDescriptor>) toanalyze_methodDescList.clone();
+
+    while (!methodDescList.isEmpty()) {
+      MethodDescriptor md = methodDescList.removeLast();
+      if (state.SSJAVADEBUG) {
+        System.out.println();
+        System.out.println("SSJAVA: Constructing a flow graph2: " + md);
+
         System.out.println("##constructSubGlobalFlowGraph");
-        GlobalFlowGraph subGlobalFlowGraph = constructSubGlobalFlowGraph(fg);
+        GlobalFlowGraph subGlobalFlowGraph = constructSubGlobalFlowGraph(getFlowGraph(md));
         mapMethodDescriptorToSubGlobalFlowGraph.put(md, subGlobalFlowGraph);
-        
+
         // TODO
         System.out.println("##addValueFlowsFromCalleeSubGlobalFlowGraph");
         addValueFlowsFromCalleeSubGlobalFlowGraph(md, subGlobalFlowGraph);
         subGlobalFlowGraph.writeGraph("_SUBGLOBAL");
 
         propagateFlowsFromCalleesWithNoCompositeLocation(md);
-        // assignCompositeLocation(md);
 
       }
     }
-    _debug_printGraph();
 
   }
 
@@ -2988,7 +3433,8 @@ public class LocationInference {
     newImplicitTupleSet.addTupleSet(condTupleNode);
 
     if (newImplicitTupleSet.size() > 1) {
-      // need to create an intermediate node for the GLB of conditional locations & implicit flows
+      // need to create an intermediate node for the GLB of conditional
+      // locations & implicit flows
       NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
       for (Iterator<NTuple<Descriptor>> idxIter = newImplicitTupleSet.iterator(); idxIter.hasNext();) {
         NTuple<Descriptor> tuple = idxIter.next();
@@ -3022,16 +3468,21 @@ public class LocationInference {
       FlowGraph fg = getFlowGraph(md);
 
       // if (implicitFlowTupleSet.size() == 1
-      // && fg.getFlowNode(implicitFlowTupleSet.iterator().next()).isIntermediate()) {
+      // &&
+      // fg.getFlowNode(implicitFlowTupleSet.iterator().next()).isIntermediate())
+      // {
       //
-      // // since there is already an intermediate node for the GLB of implicit flows
+      // // since there is already an intermediate node for the GLB of implicit
+      // flows
       // // we don't need to create another intermediate node.
       // // just re-use the intermediate node for implicit flows.
       //
-      // FlowNode meetNode = fg.getFlowNode(implicitFlowTupleSet.iterator().next());
+      // FlowNode meetNode =
+      // fg.getFlowNode(implicitFlowTupleSet.iterator().next());
       //
       // for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
-      // NTuple<Descriptor> returnNodeTuple = (NTuple<Descriptor>) iterator.next();
+      // NTuple<Descriptor> returnNodeTuple = (NTuple<Descriptor>)
+      // iterator.next();
       // fg.addValueFlowEdge(returnNodeTuple, meetNode.getDescTuple());
       // }
       //
@@ -3076,7 +3527,8 @@ public class LocationInference {
       newImplicitTupleSet.addTupleSet(condTupleNode);
 
       if (newImplicitTupleSet.size() > 1) {
-        // need to create an intermediate node for the GLB of conditional locations & implicit flows
+        // need to create an intermediate node for the GLB of conditional
+        // locations & implicit flows
         NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
         for (Iterator<NTuple<Descriptor>> idxIter = newImplicitTupleSet.iterator(); idxIter
             .hasNext();) {
@@ -3090,14 +3542,17 @@ public class LocationInference {
 
       // ///////////
       // System.out.println("condTupleNode="+condTupleNode);
-      // NTuple<Descriptor> interTuple = getFlowGraph(md).createIntermediateNode().getDescTuple();
+      // NTuple<Descriptor> interTuple =
+      // getFlowGraph(md).createIntermediateNode().getDescTuple();
       //
-      // for (Iterator<NTuple<Descriptor>> idxIter = condTupleNode.iterator(); idxIter.hasNext();) {
+      // for (Iterator<NTuple<Descriptor>> idxIter = condTupleNode.iterator();
+      // idxIter.hasNext();) {
       // NTuple<Descriptor> tuple = idxIter.next();
       // addFlowGraphEdge(md, tuple, interTuple);
       // }
 
-      // for (Iterator<NTuple<Descriptor>> idxIter = implicitFlowTupleSet.iterator(); idxIter
+      // for (Iterator<NTuple<Descriptor>> idxIter =
+      // implicitFlowTupleSet.iterator(); idxIter
       // .hasNext();) {
       // NTuple<Descriptor> tuple = idxIter.next();
       // addFlowGraphEdge(md, tuple, interTuple);
@@ -3382,6 +3837,8 @@ public class LocationInference {
 
     System.out.println("analyzeFlowMethodInvokeNode=" + min.printNode(0));
 
+    mapMethodInvokeNodeToArgIdxMap.put(min, new HashMap<Integer, NTuple<Descriptor>>());
+
     if (nodeSet == null) {
       nodeSet = new NodeTupleSet();
     }
@@ -3402,8 +3859,6 @@ public class LocationInference {
       FlowGraph calleeFlowGraph = getFlowGraph(calleeMethodDesc);
       Set<FlowNode> calleeReturnSet = calleeFlowGraph.getReturnNodeSet();
 
-      System.out.println("#calleeReturnSet=" + calleeReturnSet);
-
       if (min.getExpression() != null) {
 
         NodeTupleSet baseNodeSet = new NodeTupleSet();
@@ -3470,11 +3925,14 @@ public class LocationInference {
               addFlowGraphEdge(md, tuple, interTuple);
             }
             argTuple = interTuple;
-          } else {
+          } else if (argTupleSet.size() == 1) {
             argTuple = argTupleSet.iterator().next();
+          } else {
+            argTuple = new NTuple<Descriptor>();
           }
 
           addArgIdxMap(min, idx, argTuple);
+
           FlowNode paramNode = calleeFlowGraph.getParamFlowNode(idx);
           if (hasInFlowTo(calleeFlowGraph, paramNode, calleeReturnSet)
               || calleeMethodDesc.getModifiers().isNative()) {