import java.io.IOException;
import java.util.ArrayList;
+import java.util.Collection;
import java.util.Collections;
import java.util.Comparator;
import java.util.HashMap;
import IR.Tree.SubBlockNode;
import IR.Tree.SwitchStatementNode;
import IR.Tree.TertiaryNode;
+import IR.Tree.TreeNode;
public class LocationInference {
private Map<MethodDescriptor, Set<MethodDescriptor>> mapMethodDescToPossibleMethodDescSet;
+ public static final String GLOBALLOC = "GLOBALLOC";
+
+ public static final String TOPLOC = "TOPLOC";
+
+ public static final Descriptor GLOBALDESC = new NameDescriptor(GLOBALLOC);
+
+ public static final Descriptor TOPDESC = new NameDescriptor(TOPLOC);
+
boolean debug = true;
public LocationInference(SSJavaAnalysis ssjava, State state) {
// 2) construct lattices
inferLattices();
+ simplifyLattices();
+
debug_writeLatticeDotFile();
// 3) check properties
}
+ private void simplifyLattices() {
+
+ // generate lattice dot file
+ setupToAnalyze();
+
+ while (!toAnalyzeIsEmpty()) {
+ ClassDescriptor cd = toAnalyzeNext();
+
+ setupToAnalazeMethod(cd);
+
+ SSJavaLattice<String> classLattice = cd2lattice.get(cd);
+ if (classLattice != null) {
+ classLattice.removeRedundantEdges();
+ }
+
+ while (!toAnalyzeMethodIsEmpty()) {
+ MethodDescriptor md = toAnalyzeMethodNext();
+ if (ssjava.needTobeAnnotated(md)) {
+ SSJavaLattice<String> methodLattice = md2lattice.get(md);
+ if (methodLattice != null) {
+ methodLattice.removeRedundantEdges();
+ }
+ }
+ }
+ }
+
+ }
+
private void checkLattices() {
LinkedList<MethodDescriptor> descriptorListToAnalyze = ssjava.getSortedDescriptors();
SSJavaLattice<String> classLattice = cd2lattice.get(cd);
if (classLattice != null) {
ssjava.writeLatticeDotFile(cd, null, classLattice);
+ debug_printDescriptorToLocNameMapping(cd);
}
while (!toAnalyzeMethodIsEmpty()) {
SSJavaLattice<String> methodLattice = md2lattice.get(md);
if (methodLattice != null) {
ssjava.writeLatticeDotFile(cd, md, methodLattice);
+ debug_printDescriptorToLocNameMapping(md);
}
}
}
}
+ private void debug_printDescriptorToLocNameMapping(Descriptor desc) {
+
+ LocationInfo info = getLocationInfo(desc);
+ System.out.println("## " + desc + " ##");
+ System.out.println(info.getMapDescToInferLocation());
+ LocationInfo locInfo = getLocationInfo(desc);
+ System.out.println("mapping=" + locInfo.getMapLocSymbolToDescSet());
+ System.out.println("###################");
+
+ }
+
private void inferLattices() {
// do fixed-point analysis
LinkedList<MethodDescriptor> descriptorListToAnalyze = ssjava.getSortedDescriptors();
+ Collections.sort(descriptorListToAnalyze, new Comparator<MethodDescriptor>() {
+ public int compare(MethodDescriptor o1, MethodDescriptor o2) {
+ return o1.getSymbol().compareToIgnoreCase(o2.getSymbol());
+ }
+ });
+
// current descriptors to visit in fixed-point interprocedural analysis,
// prioritized by
// dependency in the call graph
SSJavaLattice<String> methodLattice =
new SSJavaLattice<String>(SSJavaAnalysis.TOP, SSJavaAnalysis.BOTTOM);
+ MethodLocationInfo methodInfo = new MethodLocationInfo(md);
+
System.out.println();
System.out.println("SSJAVA: Inferencing the lattice from " + md);
- analyzeMethodLattice(md, methodLattice);
+ analyzeMethodLattice(md, methodLattice, methodInfo);
SSJavaLattice<String> prevMethodLattice = getMethodLattice(md);
+ MethodLocationInfo prevMethodInfo = getMethodLocationInfo(md);
- if (!methodLattice.equals(prevMethodLattice)) {
+ if ((!methodLattice.equals(prevMethodLattice)) || (!methodInfo.equals(prevMethodInfo))) {
setMethodLattice(md, methodLattice);
+ setMethodLocInfo(md, methodInfo);
// results for callee changed, so enqueue dependents caller for
// further analysis
}
}
+ }
+ private void setMethodLocInfo(MethodDescriptor md, MethodLocationInfo methodInfo) {
+ mapMethodDescToMethodLocationInfo.put(md, methodInfo);
}
private void checkLatticesOfVirtualMethods(MethodDescriptor md) {
private void checkConsistency(MethodDescriptor md1, MethodDescriptor md2) {
// check that two lattice have the same relations between parameters(+PC
- // LOC, RETURN LOC)
+ // LOC, GLOBAL_LOC RETURN LOC)
- MethodLocationInfo methodInfo1 = getMethodLocationInfo(md1);
+ List<CompositeLocation> list1 = new ArrayList<CompositeLocation>();
+ List<CompositeLocation> list2 = new ArrayList<CompositeLocation>();
- SSJavaLattice<String> lattice1 = getMethodLattice(md1);
- SSJavaLattice<String> lattice2 = getMethodLattice(md2);
+ MethodLocationInfo locInfo1 = getMethodLocationInfo(md1);
+ MethodLocationInfo locInfo2 = getMethodLocationInfo(md2);
- Set<String> paramLocNameSet1 = methodInfo1.getParameterLocNameSet();
+ Map<Integer, CompositeLocation> paramMap1 = locInfo1.getMapParamIdxToInferLoc();
+ Map<Integer, CompositeLocation> paramMap2 = locInfo2.getMapParamIdxToInferLoc();
- for (Iterator iterator = paramLocNameSet1.iterator(); iterator.hasNext();) {
- String locName1 = (String) iterator.next();
- for (Iterator iterator2 = paramLocNameSet1.iterator(); iterator2.hasNext();) {
- String locName2 = (String) iterator2.next();
+ int numParam = locInfo1.getMapParamIdxToInferLoc().keySet().size();
- // System.out.println("COMPARE " + locName1 + " - " + locName2 + " "
- // + lattice1.isGreaterThan(locName1, locName2) + "-"
- // + lattice2.isGreaterThan(locName1, locName2));
+ // add location types of paramters
+ for (int idx = 0; idx < numParam; idx++) {
+ list1.add(paramMap1.get(Integer.valueOf(idx)));
+ list2.add(paramMap2.get(Integer.valueOf(idx)));
+ }
- if (!locName1.equals(locName2)) {
+ // add program counter location
+ list1.add(locInfo1.getPCLoc());
+ list2.add(locInfo2.getPCLoc());
+
+ if (!md1.getReturnType().isVoid()) {
+ // add return value location
+ CompositeLocation rtrLoc1 =
+ new CompositeLocation(new Location(md1, locInfo1.getReturnLocName()));
+ CompositeLocation rtrLoc2 =
+ new CompositeLocation(new Location(md2, locInfo2.getReturnLocName()));
+ list1.add(rtrLoc1);
+ list2.add(rtrLoc2);
+ }
+
+ // add global location type
+ if (md1.isStatic()) {
+ CompositeLocation globalLoc1 =
+ new CompositeLocation(new Location(md1, locInfo1.getGlobalLocName()));
+ CompositeLocation globalLoc2 =
+ new CompositeLocation(new Location(md2, locInfo2.getGlobalLocName()));
+ list1.add(globalLoc1);
+ list2.add(globalLoc2);
+ }
+
+ for (int i = 0; i < list1.size(); i++) {
+ CompositeLocation locA1 = list1.get(i);
+ CompositeLocation locA2 = list2.get(i);
+ for (int k = 0; k < list1.size(); k++) {
+ if (i != k) {
+ CompositeLocation locB1 = list1.get(k);
+ CompositeLocation locB2 = list2.get(k);
+ boolean r1 = isGreaterThan(locA1, locB1);
- boolean r1 = lattice1.isGreaterThan(locName1, locName2);
- boolean r2 = lattice2.isGreaterThan(locName1, locName2);
+ boolean r2 = isGreaterThan(locA2, locB2);
if (r1 != r2) {
throw new Error("The method " + md1 + " is not consistent with the method " + md2
- + ".:: They have a different ordering relation between parameters " + locName1
- + " and " + locName2 + ".");
+ + ".:: They have a different ordering relation between locations (" + locA1 + ","
+ + locB1 + ") and (" + locA2 + "," + locB2 + ").");
}
}
-
}
}
return desc;
}
- private void analyzeMethodLattice(MethodDescriptor md, SSJavaLattice<String> methodLattice) {
-
- MethodLocationInfo methodInfo = getMethodLocationInfo(md);
+ private void analyzeMethodLattice(MethodDescriptor md, SSJavaLattice<String> methodLattice,
+ MethodLocationInfo methodInfo) {
// first take a look at method invocation nodes to newly added relations
// from the callee
analyzeLatticeMethodInvocationNode(md);
+ // set the this location
+ String thisLocSymbol = md.getThis().getSymbol();
+ methodInfo.setThisLocName(thisLocSymbol);
+
+ // set the global location
+ methodInfo.setGlobalLocName(LocationInference.GLOBALLOC);
+
// visit each node of method flow graph
FlowGraph fg = getFlowGraph(md);
Set<FlowNode> nodeSet = fg.getNodeSet();
ClassDescriptor varClassDesc = varDesc.getType().getClassDesc();
extractRelationFromFieldFlows(varClassDesc, srcNode, dstNode, 1);
- } else {
+ } else if (srcNodeTuple.size() == 1 || dstNodeTuple.size() == 1) {
// for the method lattice, we need to look at the first element of
// NTuple<Descriptor>
- if (srcNodeTuple.size() == 1 || dstNodeTuple.size() == 1) {
+ // in this case, take a look at connected nodes at the local level
+ addRelationToLattice(md, methodLattice, methodInfo, srcNode, dstNode);
+ } else {
+
+ if (!srcNode.getDescTuple().get(0).equals(dstNode.getDescTuple().get(0))) {
// in this case, take a look at connected nodes at the local level
- addRelationToLattice(md, methodLattice, srcNode, dstNode);
+ addRelationToLattice(md, methodLattice, methodInfo, srcNode, dstNode);
+ } else {
+ Descriptor srcDesc = srcNode.getDescTuple().get(0);
+ Descriptor dstDesc = dstNode.getDescTuple().get(0);
+ recursivelyAddCompositeRelation(md, fg, methodInfo, srcNode, dstNode, srcDesc,
+ dstDesc);
+ // recursiveAddRelationToLattice(1, md, srcNode, dstNode);
}
}
}
}
- // grab the this location if the method use the 'this' reference
- String thisLocSymbol = md.getThis().getSymbol();
- if (methodLattice.getKeySet().contains(thisLocSymbol)) {
- methodInfo.setThisLocName(thisLocSymbol);
+ // create mapping from param idx to inferred composite location
+
+ int offset;
+ if (!md.isStatic()) {
+ // add 'this' reference location
+ offset = 1;
+ methodInfo.addMapParamIdxToInferLoc(0, methodInfo.getInferLocation(md.getThis()));
+ } else {
+ offset = 0;
+ }
+
+ for (int idx = 0; idx < md.numParameters(); idx++) {
+ Descriptor paramDesc = md.getParameter(idx);
+ CompositeLocation inferParamLoc = methodInfo.getInferLocation(paramDesc);
+ methodInfo.addMapParamIdxToInferLoc(idx + offset, inferParamLoc);
+ }
+
+ // calculate the initial program counter location
+ // PC location is higher than location types of all parameters
+ String pcLocSymbol = "PCLOC";
+ Map<Integer, CompositeLocation> mapParamToLoc = methodInfo.getMapParamIdxToInferLoc();
+ Set<Integer> keySet = mapParamToLoc.keySet();
+ for (Iterator iterator = keySet.iterator(); iterator.hasNext();) {
+ Integer paramIdx = (Integer) iterator.next();
+ CompositeLocation inferLoc = mapParamToLoc.get(paramIdx);
+ String paramLocLocalSymbol = inferLoc.get(0).getLocIdentifier();
+ if (!methodLattice.isGreaterThan(pcLocSymbol, paramLocLocalSymbol)) {
+ addRelationHigherToLower(methodLattice, methodInfo, pcLocSymbol, paramLocLocalSymbol);
+ }
}
// calculate a return location
for (Iterator iterator = returnNodeSet.iterator(); iterator.hasNext();) {
FlowNode rtrNode = (FlowNode) iterator.next();
- String localSymbol = rtrNode.getDescTuple().get(0).getSymbol();
+ String localSymbol =
+ methodInfo.getInferLocation(rtrNode.getDescTuple().get(0)).get(0).getLocIdentifier();
returnVarSymbolSet.add(localSymbol);
}
String returnGLB = methodLattice.getGLB(returnVarSymbolSet);
if (returnGLB.equals(SSJavaAnalysis.BOTTOM)) {
- // need to insert a new location in-between the bottom and all locations
+ // need to insert a new location in-between the bottom and all
+ // locations
// that is directly connected to the bottom
String returnNewLocationSymbol = "Loc" + (SSJavaLattice.seed++);
methodLattice.insertNewLocationAtOneLevelHigher(returnGLB, returnNewLocationSymbol);
}
+ private CompositeLocation getHighestLocation(Collection<CompositeLocation> locSet) {
+
+ Iterator<CompositeLocation> locIter = locSet.iterator();
+
+ CompositeLocation highest = locIter.next();
+
+ for (; locIter.hasNext();) {
+ CompositeLocation loc = (CompositeLocation) locIter.next();
+ if (isGreaterThan(loc, highest)) {
+ highest = loc;
+ }
+ }
+
+ return highest;
+
+ }
+
+ private boolean isGreaterThan(CompositeLocation comp1, CompositeLocation comp2) {
+
+ for (int idx = 0; idx < comp1.getSize(); idx++) {
+ Location loc1 = comp1.get(idx);
+ Location loc2 = comp2.get(idx);
+
+ Descriptor desc1 = loc1.getDescriptor();
+ Descriptor desc2 = loc2.getDescriptor();
+
+ if (!desc1.equals(desc2)) {
+ throw new Error("Fail to compare " + comp1 + " and " + comp2);
+ }
+
+ String symbol1 = loc1.getLocIdentifier();
+ String symbol2 = loc2.getLocIdentifier();
+
+ if (symbol1.equals(symbol2)) {
+ continue;
+ } else if (getLattice(desc1).isGreaterThan(symbol1, symbol2)) {
+ return true;
+ } else {
+ return false;
+ }
+
+ }
+
+ return false;
+ }
+
+ private void recursiveAddRelationToLattice(int idx, MethodDescriptor md,
+ CompositeLocation srcInferLoc, CompositeLocation dstInferLoc) {
+
+ String srcLocSymbol = srcInferLoc.get(idx).getLocIdentifier();
+ String dstLocSymbol = dstInferLoc.get(idx).getLocIdentifier();
+
+ if (srcLocSymbol.equals(dstLocSymbol)) {
+ recursiveAddRelationToLattice(idx + 1, md, srcInferLoc, dstInferLoc);
+ } else {
+
+ Descriptor parentDesc = srcInferLoc.get(idx).getDescriptor();
+ LocationInfo locInfo = getLocationInfo(parentDesc);
+
+ addRelationHigherToLower(getLattice(parentDesc), getLocationInfo(parentDesc), srcLocSymbol,
+ dstLocSymbol);
+ }
+
+ }
+
private void analyzeLatticeMethodInvocationNode(MethodDescriptor mdCaller) {
// the transformation for a call site propagates all relations between
setPossibleCallees.addAll(ssjava.getCallGraph().getMethods(mdCallee));
}
+ System.out.println("mdCaller=" + mdCaller + " setPossibleCallees=" + setPossibleCallees);
for (Iterator iterator2 = setPossibleCallees.iterator(); iterator2.hasNext();) {
MethodDescriptor possibleMdCallee = (MethodDescriptor) iterator2.next();
propagateRelationToCaller(min, mdCaller, possibleMdCallee);
FlowGraph calleeFlowGraph = getFlowGraph(possibleMdCallee);
+ System.out.println("calleeFlowGraph=" + calleeFlowGraph + " of " + possibleMdCallee);
// find parameter node
Set<FlowNode> paramNodeSet = calleeFlowGraph.getParameterNodeSet();
NTuple<Descriptor> higherArg = getArgTupleByArgIdx(min, higherLocIdxCallee);
NTuple<Descriptor> lowerArg = getArgTupleByArgIdx(min, lowerLocIdxCallee);
- addFlowGraphEdge(mdCaller, higherArg, lowerArg);
+ if (higherArg != null && lowerArg != null) {
+ // if the argument has the TOP location, getArgTupleByArgIdx returns
+ // null
+ addFlowGraphEdge(mdCaller, higherArg, lowerArg);
+ }
}
}
private void addRelationToLattice(MethodDescriptor md, SSJavaLattice<String> methodLattice,
- FlowNode srcNode, FlowNode dstNode) {
+ MethodLocationInfo methodInfo, FlowNode srcNode, FlowNode dstNode) {
+ System.out.println();
System.out.println("### addRelationToLattice src=" + srcNode + " dst=" + dstNode);
// add a new binary relation of dstNode < srcNode
FlowGraph flowGraph = getFlowGraph(md);
- MethodLocationInfo methodInfo = getMethodLocationInfo(md);
-
- String srcOriginSymbol = getSymbol(0, srcNode);
- String dstOriginSymbol = getSymbol(0, dstNode);
Descriptor srcDesc = getDescriptor(0, srcNode);
Descriptor dstDesc = getDescriptor(0, dstNode);
- String srcSymbol = methodInfo.getLocName(srcOriginSymbol);
- String dstSymbol = methodInfo.getLocName(dstOriginSymbol);
-
- if (srcNode.isParameter()) {
- int paramIdx = flowGraph.getParamIdx(srcNode.getDescTuple());
- methodInfo.addParameter(srcSymbol, srcDesc, paramIdx);
- } else {
- methodInfo.addMappingOfLocNameToDescriptor(srcSymbol, srcDesc);
- }
-
- if (dstNode.isParameter()) {
- int paramIdx = flowGraph.getParamIdx(dstNode.getDescTuple());
- methodInfo.addParameter(dstSymbol, dstDesc, paramIdx);
- } else {
- methodInfo.addMappingOfLocNameToDescriptor(dstSymbol, dstDesc);
- }
-
- // consider a composite location case
+ // boolean isAssignedCompositeLocation = false;
+ // if (!methodInfo.getInferLocation(srcDesc).get(0).getLocIdentifier()
+ // .equals(methodInfo.getThisLocName())) {
+ // isAssignedCompositeLocation =
+ calculateCompositeLocation(flowGraph, methodLattice, methodInfo, srcNode);
+ // }
- boolean isSrcLocalVar = false;
- boolean isDstLocalVar = false;
- if (srcNode.getDescTuple().size() == 1) {
- isSrcLocalVar = true;
- }
+ String srcSymbol = methodInfo.getInferLocation(srcDesc).get(0).getLocIdentifier();
+ String dstSymbol = methodInfo.getInferLocation(dstDesc).get(0).getLocIdentifier();
- if (dstNode.getDescTuple().size() == 1) {
- isDstLocalVar = true;
- }
+ // if (srcNode.isParameter()) {
+ // int paramIdx = flowGraph.getParamIdx(srcNode.getDescTuple());
+ // methodInfo.addParameter(srcSymbol, srcDesc, paramIdx);
+ // }
+ //
+ // if (dstNode.isParameter()) {
+ // int paramIdx = flowGraph.getParamIdx(dstNode.getDescTuple());
+ // methodInfo.addParameter(dstSymbol, dstDesc, paramIdx);
+ // }
- boolean isAssignedCompositeLocation = false;
- if (isSrcLocalVar && isDstLocalVar) {
- // both src and dst are local var
+ CompositeLocation srcInferLoc = methodInfo.getInferLocation(srcDesc);
+ CompositeLocation dstInferLoc = methodInfo.getInferLocation(dstDesc);
- // CompositeLocation inferSrc=methodInfo.getInferLocation(srcNode);
- // CompositeLocation inferDst=methodInfo.getInferLocation(dstNode);
- // if( (inferSrc!=null && inferSrc.getSize()>1) || (inferDst!=null &&
- // inferDst.getSize()>1)){
- // isAssignedCompositeLocation = true;
- // }
+ String srcLocalLocSymbol = srcInferLoc.get(0).getLocIdentifier();
+ String dstLocalLocSymbol = dstInferLoc.get(0).getLocIdentifier();
- // TODO: need to fix
- isAssignedCompositeLocation =
- calculateCompositeLocationForLocalVar(flowGraph, methodLattice, methodInfo, srcNode);
- calculateCompositeLocationForLocalVar(flowGraph, methodLattice, methodInfo, dstNode);
-
- } else if (isSrcLocalVar) {
- // src is local var
- isAssignedCompositeLocation =
- calculateCompositeLocationForLocalVar(flowGraph, methodLattice, methodInfo, srcNode);
- } else if (isDstLocalVar) {
- // dst is local var
- isAssignedCompositeLocation =
- calculateCompositeLocationForLocalVar(flowGraph, methodLattice, methodInfo, dstNode);
- }
-
- if (!isAssignedCompositeLocation) {
- if (!methodLattice.isGreaterThan(srcSymbol, dstSymbol)) {
- // if the lattice does not have this relation, add it
- methodLattice.addRelationHigherToLower(srcSymbol, dstSymbol);
+ if (srcInferLoc.getSize() == 1 && dstInferLoc.getSize() == 1) {
+ // add a new relation to the local lattice
+ addRelationHigherToLower(methodLattice, methodInfo, srcLocalLocSymbol, dstLocalLocSymbol);
+ } else if (srcInferLoc.getSize() > 1 && dstInferLoc.getSize() > 1) {
+ // both src and dst have assigned to a composite location
+ recursivelyAddRelation(1, srcInferLoc, dstInferLoc);
+ } else {
+ // either src or dst has assigned to a composite location
+ if (!srcLocalLocSymbol.equals(dstLocalLocSymbol)) {
+ addRelationHigherToLower(methodLattice, methodInfo, srcLocalLocSymbol, dstLocalLocSymbol);
}
}
-
- // Set<String> cycleElementSet =
- // methodLattice.getPossibleCycleElements(srcSymbol, dstSymbol);
+ // if (!isAssignedCompositeLocation) {
+ // // source does not have a composite location
//
- // System.out.println("### POSSIBLE CYCLE ELEMENT SET=" + cycleElementSet);
- // boolean hasNonPrimitiveElement = false;
- // for (Iterator iterator = cycleElementSet.iterator(); iterator.hasNext();)
- // {
- // String cycleElementSymbol = (String) iterator.next();
- // Set<Descriptor> flowNodeSet =
- // methodInfo.getFlowNodeSet(cycleElementSymbol);
- // for (Iterator iterator2 = flowNodeSet.iterator(); iterator2.hasNext();) {
- // Descriptor desc = (Descriptor) iterator2.next();
- // if (!isPrimitiveTypeDescriptor(desc)) {
- // hasNonPrimitiveElement = true;
- // }
- // System.out
- // .println("flowNode=" + desc + " is primitive?=" +
- // isPrimitiveTypeDescriptor(desc));
- // }
- // }
+ // NTuple<Location> srcTuple = flowGraph.getLocationTuple(srcNode);
+ // NTuple<Location> dstTuple = flowGraph.getLocationTuple(dstNode);
//
- // if (hasNonPrimitiveElement) {
- // // if there is non-primitive element in the cycle, no way to merge cyclic
- // // elements into the shared location
- // System.out.println("Failed to merge cyclic value flows into a shared location.");
+ // recursivelyAddCompositeRelation(md, flowGraph, methodInfo, srcNode,
+ // dstNode, srcDesc, dstDesc);
//
- // // try to assign more fine-grind composite location to the source or the
- // // destination
- // System.out.println("SRC=" + srcSymbol + " DST=" + dstSymbol);
- // System.out.println("### SRCNODE=" + srcNode + " DSTNODE=" + dstNode);
+ // // if (!srcSymbol.equals(dstSymbol)) {
+ // // // add a local relation
+ // // if (!methodLattice.isGreaterThan(srcSymbol, dstSymbol)) {
+ // // // if the lattice does not have this relation, add it
+ // // addRelationHigherToLower(methodLattice, methodInfo, srcSymbol,
+ // // dstSymbol);
+ // // // methodLattice.addRelationHigherToLower(srcSymbol, dstSymbol);
+ // // }
+ // // } else {
+ // // // if src and dst have the same local location...
+ // // recursivelyAddCompositeRelation(md, flowGraph, methodInfo, srcNode,
+ // // dstNode, srcDesc,
+ // // dstDesc);
+ // // }
//
- // FlowNode targetNode;
- // if (isPrimitiveLocalVariable(srcNode)) {
- // targetNode = srcNode;
// } else {
- // targetNode = dstNode;
- // }
- //
- // calculateMoreFineGrainedLocation(md, methodLattice, targetNode);
- //
- // return;
+ // // source variable has a composite location
+ // if (methodInfo.getInferLocation(dstDesc).getSize() == 1) {
+ // if (!srcSymbol.equals(dstSymbol)) {
+ // addRelationHigherToLower(methodLattice, methodInfo, srcSymbol,
+ // dstSymbol);
// }
- //
- // if (cycleElementSet.size() > 0) {
- // String newSharedLoc = "SharedLoc" + (SSJavaLattice.seed++);
- // methodLattice.mergeIntoSharedLocation(cycleElementSet, newSharedLoc);
- //
- // for (Iterator iterator = cycleElementSet.iterator(); iterator.hasNext();)
- // {
- // String locName = (String) iterator.next();
- // methodInfo.mapDescSymbolToLocName(locName, newSharedLoc);
// }
//
- // } else if (!methodLattice.isGreaterThan(srcSymbol, dstSymbol)) {
- // // if the lattice does not have this relation, add it
- // methodLattice.addRelationHigherToLower(srcSymbol, dstSymbol);
// }
- //
- // System.out.println("methodLattice=" + methodLattice.getKeySet());
}
- private void addPrefixMapping(Map<NTuple<Location>, Set<NTuple<Location>>> map,
- NTuple<Location> prefix, NTuple<Location> element) {
+ private void recursivelyAddRelation(int idx, CompositeLocation srcInferLoc,
+ CompositeLocation dstInferLoc) {
- if (!map.containsKey(prefix)) {
- map.put(prefix, new HashSet<NTuple<Location>>());
+ String srcLocSymbol = srcInferLoc.get(idx).getLocIdentifier();
+ String dstLocSymbol = dstInferLoc.get(idx).getLocIdentifier();
+
+ if (srcLocSymbol.equals(dstLocSymbol)) {
+ recursivelyAddRelation(idx + 1, srcInferLoc, dstInferLoc);
+ } else {
+
+ Descriptor parentDesc = srcInferLoc.get(idx).getDescriptor();
+
+ addRelationHigherToLower(getLattice(parentDesc), getLocationInfo(parentDesc), srcLocSymbol,
+ dstLocSymbol);
}
- map.get(prefix).add(element);
}
- private NTuple<Location> getLocationTuple(MethodLocationInfo methodInfo, FlowGraph flowGraph,
- FlowNode flowNode) {
+ private void recursivelyAddCompositeRelation(MethodDescriptor md, FlowGraph flowGraph,
+ MethodLocationInfo methodInfo, FlowNode srcNode, FlowNode dstNode, Descriptor srcDesc,
+ Descriptor dstDesc) {
+
+ CompositeLocation inferSrcLoc;
+ CompositeLocation inferDstLoc = methodInfo.getInferLocation(dstDesc);
+
+ if (srcNode.getDescTuple().size() > 1) {
+ // field access
+ inferSrcLoc = new CompositeLocation();
- NTuple<Location> locTuple;
- CompositeLocation inferLoc = methodInfo.getInferLocation(flowNode);
- if (inferLoc != null) {
- // the flow node has already been assigned to the location
- locTuple = new NTuple<Location>();
- NTuple<Location> inferLocTuple = inferLoc.getTuple();
- for (int i = 0; i < inferLocTuple.size(); i++) {
- locTuple.add(inferLocTuple.get(i));
+ NTuple<Location> locTuple = flowGraph.getLocationTuple(srcNode);
+ for (int i = 0; i < locTuple.size(); i++) {
+ inferSrcLoc.addLocation(locTuple.get(i));
}
+
} else {
- locTuple = flowGraph.getLocationTuple(flowNode);
+ inferSrcLoc = methodInfo.getInferLocation(srcDesc);
}
- return locTuple;
+ if (dstNode.getDescTuple().size() > 1) {
+ // field access
+ inferDstLoc = new CompositeLocation();
+
+ NTuple<Location> locTuple = flowGraph.getLocationTuple(dstNode);
+ for (int i = 0; i < locTuple.size(); i++) {
+ inferDstLoc.addLocation(locTuple.get(i));
+ }
+
+ } else {
+ inferDstLoc = methodInfo.getInferLocation(dstDesc);
+ }
+
+ recursiveAddRelationToLattice(1, md, inferSrcLoc, inferDstLoc);
}
- private boolean calculateCompositeLocationForLocalVar(FlowGraph flowGraph,
+ private void addPrefixMapping(Map<NTuple<Location>, Set<NTuple<Location>>> map,
+ NTuple<Location> prefix, NTuple<Location> element) {
+
+ if (!map.containsKey(prefix)) {
+ map.put(prefix, new HashSet<NTuple<Location>>());
+ }
+ map.get(prefix).add(element);
+ }
+
+ private boolean calculateCompositeLocation(FlowGraph flowGraph,
SSJavaLattice<String> methodLattice, MethodLocationInfo methodInfo, FlowNode flowNode) {
+ Descriptor localVarDesc = flowNode.getDescTuple().get(0);
+
Set<FlowNode> inNodeSet = flowGraph.getIncomingFlowNodeSet(flowNode);
Set<FlowNode> reachableNodeSet = flowGraph.getReachableFlowNodeSet(flowNode);
Map<NTuple<Location>, Set<NTuple<Location>>> mapPrefixToIncomingLocTupleSet =
new HashMap<NTuple<Location>, Set<NTuple<Location>>>();
+ Set<FlowNode> localInNodeSet = new HashSet<FlowNode>();
+ Set<FlowNode> localOutNodeSet = new HashSet<FlowNode>();
+
List<NTuple<Location>> prefixList = new ArrayList<NTuple<Location>>();
for (Iterator iterator = inNodeSet.iterator(); iterator.hasNext();) {
FlowNode inNode = (FlowNode) iterator.next();
- NTuple<Location> inTuple = getLocationTuple(methodInfo, flowGraph, inNode);
+ NTuple<Location> inTuple = flowGraph.getLocationTuple(inNode);
if (inTuple.size() > 1) {
for (int i = 1; i < inTuple.size(); i++) {
}
addPrefixMapping(mapPrefixToIncomingLocTupleSet, prefix, inTuple);
}
+ } else {
+ localInNodeSet.add(inNode);
}
}
}
});
+ for (Iterator iterator2 = reachableNodeSet.iterator(); iterator2.hasNext();) {
+ FlowNode reachableNode = (FlowNode) iterator2.next();
+ if (reachableNode.getDescTuple().size() == 1) {
+ localOutNodeSet.add(reachableNode);
+ }
+ }
+
// find out reachable nodes that have the longest common prefix
for (int i = 0; i < prefixList.size(); i++) {
NTuple<Location> curPrefix = prefixList.get(i);
if (reachLocTuple.startsWith(curPrefix)) {
reachableCommonPrefixSet.add(reachLocTuple);
}
+
}
if (!reachableCommonPrefixSet.isEmpty()) {
+ // found reachable nodes that start with the prefix curPrefix
+ // need to assign a composite location
+
+ // first, check if there are more than one the set of locations that has
+ // the same length of the longest reachable prefix, no way to assign
+ // a composite location to the input local var
+ prefixSanityCheck(prefixList, i, flowGraph, reachableNodeSet);
+
Set<NTuple<Location>> incomingCommonPrefixSet =
mapPrefixToIncomingLocTupleSet.get(curPrefix);
SSJavaLattice<String> lattice = getLattice(desc);
LocationInfo locInfo = getLocationInfo(desc);
- CompositeLocation inferNode = methodInfo.getInferLocation(flowNode);
- String nodeSymbol;
- if (inferNode != null) {
+ // CompositeLocation inferLocation =
+ // methodInfo.getInferLocation(flowNode);
+ CompositeLocation inferLocation = methodInfo.getInferLocation(localVarDesc);
+
+ String newlyInsertedLocName;
+ if (inferLocation.getSize() == 1) {
+ // need to replace the old local location with a new composite
+ // location
+
+ String oldMethodLocationSymbol = inferLocation.get(0).getLocIdentifier();
- } else {
String newLocSymbol = "Loc" + (SSJavaLattice.seed++);
- inferNode = new CompositeLocation();
+ inferLocation = new CompositeLocation();
for (int locIdx = 0; locIdx < curPrefix.size(); locIdx++) {
- inferNode.addLocation(curPrefix.get(locIdx));
- }
- inferNode.addLocation(new Location(desc, newLocSymbol));
- methodInfo.mapFlowNodeToInferLocation(flowNode, inferNode);
- if (flowNode.getDescTuple().size() == 1) {
- // local variable case
- modifyLocalLatticeAccrodingToNewlyAddedCompositeLocation(methodLattice, methodInfo,
- inferNode, flowNode);
+ inferLocation.addLocation(curPrefix.get(locIdx));
}
+ Location fieldLoc = new Location(desc, newLocSymbol);
+ inferLocation.addLocation(fieldLoc);
+
+ methodInfo.mapDescriptorToLocation(localVarDesc, inferLocation);
+ methodInfo.removeMaplocalVarToLocSet(localVarDesc);
+
+ String newMethodLocationSymbol = curPrefix.get(0).getLocIdentifier();
+
+ replaceOldLocWithNewLoc(methodLattice, oldMethodLocationSymbol, newMethodLocationSymbol);
+
+ } else {
+
+ String localLocName = methodInfo.getInferLocation(localVarDesc).get(0).getLocIdentifier();
+ return true;
+
}
- nodeSymbol = inferNode.get(inferNode.getSize() - 1).getLocIdentifier();
+ newlyInsertedLocName = inferLocation.get(inferLocation.getSize() - 1).getLocIdentifier();
for (Iterator iterator = incomingCommonPrefixSet.iterator(); iterator.hasNext();) {
NTuple<Location> tuple = (NTuple<Location>) iterator.next();
+
Location loc = tuple.get(idx);
- String higher = locInfo.getLocName(loc.getLocIdentifier());
- lattice.addRelationHigherToLower(higher, nodeSymbol);
+ String higher = locInfo.getFieldInferLocation(loc.getLocDescriptor()).getLocIdentifier();
+ System.out.println("--");
+ System.out.println("add in-flow relation:");
+ addRelationHigherToLower(lattice, locInfo, higher, newlyInsertedLocName);
+ }
+ System.out.println("end of add-inflow relation");
+
+ for (Iterator iterator = localInNodeSet.iterator(); iterator.hasNext();) {
+ FlowNode localNode = (FlowNode) iterator.next();
+ Descriptor localInVarDesc = localNode.getDescTuple().get(0);
+ CompositeLocation inNodeInferLoc = methodInfo.getInferLocation(localInVarDesc);
+
+ if (isCompositeLocation(inNodeInferLoc)) {
+ // need to make sure that newLocSymbol is lower than the infernode
+ // location in the field lattice
+
+ if (inNodeInferLoc.getTuple().startsWith(curPrefix)
+ && inNodeInferLoc.getSize() == (curPrefix.size() + 1)) {
+ String higher = inNodeInferLoc.get(inNodeInferLoc.getSize() - 1).getLocIdentifier();
+ if (!higher.equals(newlyInsertedLocName)) {
+ System.out.println("add localInNodeSet relation:");
+ addRelationHigherToLower(lattice, locInfo, higher, newlyInsertedLocName);
+ }
+ } else {
+ throw new Error("Failed to generate a composite location.");
+ }
+
+ }
}
for (Iterator iterator = reachableCommonPrefixSet.iterator(); iterator.hasNext();) {
NTuple<Location> tuple = (NTuple<Location>) iterator.next();
Location loc = tuple.get(idx);
- String lower = locInfo.getLocName(loc.getLocIdentifier());
- lattice.addRelationHigherToLower(nodeSymbol, lower);
+ String lower = locInfo.getFieldInferLocation(loc.getLocDescriptor()).getLocIdentifier();
+ // lattice.addRelationHigherToLower(newlyInsertedLocName, lower);
+ System.out.println("add out-flow relation:");
+ addRelationHigherToLower(lattice, locInfo, newlyInsertedLocName, lower);
+ }
+ System.out.println("end of add out-flow relation");
+
+ for (Iterator iterator = localOutNodeSet.iterator(); iterator.hasNext();) {
+ FlowNode localOutNode = (FlowNode) iterator.next();
+
+ Descriptor localOutDesc = localOutNode.getDescTuple().get(0);
+ // String localOutNodeSymbol =
+ // localOutNode.getDescTuple().get(0).getSymbol();
+ CompositeLocation outNodeInferLoc = methodInfo.getInferLocation(localOutDesc);
+
+ // System.out
+ // .println("localOutNode=" + localOutNode + " outNodeInferLoc=" +
+ // outNodeInferLoc);
+ if (isCompositeLocation(outNodeInferLoc)) {
+ // need to make sure that newLocSymbol is higher than the infernode
+ // location
+
+ if (outNodeInferLoc.getTuple().startsWith(curPrefix)
+ && outNodeInferLoc.getSize() == (curPrefix.size() + 1)) {
+
+ String lower = outNodeInferLoc.get(outNodeInferLoc.getSize() - 1).getLocIdentifier();
+ System.out.println("add outNodeInferLoc relation:");
+
+ addRelationHigherToLower(lattice, locInfo, newlyInsertedLocName, lower);
+
+ } else {
+ throw new Error("Failed to generate a composite location.");
+ }
+ }
}
return true;
}
- private void modifyLocalLatticeAccrodingToNewlyAddedCompositeLocation(
- SSJavaLattice<String> methodLattice, MethodLocationInfo methodInfo,
- CompositeLocation inferNode, FlowNode flowNode) {
+ private boolean isCompositeLocation(CompositeLocation cl) {
+ return cl.getSize() > 1;
+ }
+
+ private boolean containsNonPrimitiveElement(Set<Descriptor> descSet) {
+ for (Iterator iterator = descSet.iterator(); iterator.hasNext();) {
+ Descriptor desc = (Descriptor) iterator.next();
+
+ if (desc.equals(LocationInference.GLOBALDESC)) {
+ return true;
+ } else if (desc instanceof VarDescriptor) {
+ if (!((VarDescriptor) desc).getType().isPrimitive()) {
+ return true;
+ }
+ } else if (desc instanceof FieldDescriptor) {
+ if (!((FieldDescriptor) desc).getType().isPrimitive()) {
+ return true;
+ }
+ }
+
+ }
+ return false;
+ }
+
+ private void addRelationHigherToLower(SSJavaLattice<String> lattice, LocationInfo locInfo,
+ String higher, String lower) {
+
+ // if (higher.equals(lower) && lattice.isSharedLoc(higher)) {
+ // return;
+ // }
+
+ Set<String> cycleElementSet = lattice.getPossibleCycleElements(higher, lower);
+ System.out.println("#Check cycle=" + lower + " < " + higher);
+ System.out.println("#cycleElementSet=" + cycleElementSet);
+
+ boolean hasNonPrimitiveElement = false;
+ for (Iterator iterator = cycleElementSet.iterator(); iterator.hasNext();) {
+ String cycleElementLocSymbol = (String) iterator.next();
+
+ Set<Descriptor> descSet = locInfo.getDescSet(cycleElementLocSymbol);
+ if (containsNonPrimitiveElement(descSet)) {
+ hasNonPrimitiveElement = true;
+ break;
+ }
+ }
+
+ if (hasNonPrimitiveElement) {
+ // if there is non-primitive element in the cycle, no way to merge cyclic
+ // elements into the shared location
+ throw new Error("Failed to merge cyclic value flows into a shared location.");
+ }
+
+ if (cycleElementSet.size() > 0) {
+ String newSharedLoc = "SharedLoc" + (SSJavaLattice.seed++);
+
+ lattice.mergeIntoSharedLocation(cycleElementSet, newSharedLoc);
+
+ for (Iterator iterator = cycleElementSet.iterator(); iterator.hasNext();) {
+ String oldLocSymbol = (String) iterator.next();
+ locInfo.mergeMapping(oldLocSymbol, newSharedLoc);
+ }
+
+ lattice.addSharedLoc(newSharedLoc);
+
+ } else if (!lattice.isGreaterThan(higher, lower)) {
+ lattice.addRelationHigherToLower(higher, lower);
+ }
+ }
+
+ private void replaceOldLocWithNewLoc(SSJavaLattice<String> methodLattice, String oldLocSymbol,
+ String newLocSymbol) {
+
+ if (methodLattice.containsKey(oldLocSymbol)) {
+ methodLattice.substituteLocation(oldLocSymbol, newLocSymbol);
+ }
+
+ }
+
+ private void prefixSanityCheck(List<NTuple<Location>> prefixList, int curIdx,
+ FlowGraph flowGraph, Set<FlowNode> reachableNodeSet) {
- Location localLocation = inferNode.get(0);
- String newLocName = methodInfo.getLocName(localLocation.getLocIdentifier());
- String oldLocName = methodInfo.getLocName(flowNode.getDescTuple().get(0).getSymbol());
+ NTuple<Location> curPrefix = prefixList.get(curIdx);
- methodInfo.mapDescSymbolToLocName(oldLocName, newLocName);
- methodLattice.substituteLocation(oldLocName, newLocName);
+ for (int i = curIdx + 1; i < prefixList.size(); i++) {
+ NTuple<Location> prefixTuple = prefixList.get(i);
+ if (curPrefix.startsWith(prefixTuple)) {
+ continue;
+ }
+
+ for (Iterator iterator2 = reachableNodeSet.iterator(); iterator2.hasNext();) {
+ FlowNode reachableNode = (FlowNode) iterator2.next();
+ NTuple<Location> reachLocTuple = flowGraph.getLocationTuple(reachableNode);
+ if (reachLocTuple.startsWith(prefixTuple)) {
+ // TODO
+ throw new Error("Failed to generate a composite location");
+ }
+ }
+ }
}
public boolean isPrimitiveLocalVariable(FlowNode node) {
// add a new binary relation of dstNode < srcNode
SSJavaLattice<String> fieldLattice = getFieldLattice(cd);
-
- String srcOriginalSymbol = srcFieldDesc.getSymbol();
- String dstOriginalSymbol = dstFieldDesc.getSymbol();
-
LocationInfo fieldInfo = getFieldLocationInfo(cd);
- String srcSymbol = fieldInfo.getLocName(srcOriginalSymbol);
- String dstSymbol = fieldInfo.getLocName(dstOriginalSymbol);
-
- Set<String> cycleElementSet = fieldLattice.getPossibleCycleElements(srcSymbol, dstSymbol);
+ String srcSymbol = fieldInfo.getFieldInferLocation(srcFieldDesc).getLocIdentifier();
+ String dstSymbol = fieldInfo.getFieldInferLocation(dstFieldDesc).getLocIdentifier();
- if (cycleElementSet.size() > 0) {
- String newSharedLoc = "SharedLoc" + (SSJavaLattice.seed++);
- fieldLattice.mergeIntoSharedLocation(cycleElementSet, newSharedLoc);
-
- for (Iterator iterator = cycleElementSet.iterator(); iterator.hasNext();) {
- String locName = (String) iterator.next();
- fieldInfo.mapDescSymbolToLocName(locName, newSharedLoc);
- }
-
- } else if (!fieldLattice.isGreaterThan(srcSymbol, dstSymbol)) {
- // if the lattice does not have this relation, add it
- fieldLattice.addRelationHigherToLower(srcSymbol, dstSymbol);
- }
+ addRelationHigherToLower(fieldLattice, fieldInfo, srcSymbol, dstSymbol);
}
public void constructFlowGraph() {
setupToAnalyze();
+
+ Set<MethodDescriptor> visited=new HashSet<MethodDescriptor>();
while (!toAnalyzeIsEmpty()) {
ClassDescriptor cd = toAnalyzeNext();
setupToAnalazeMethod(cd);
while (!toAnalyzeMethodIsEmpty()) {
MethodDescriptor md = toAnalyzeMethodNext();
- if (ssjava.needTobeAnnotated(md)) {
+// if (ssjava.needTobeAnnotated(md)) {
if (state.SSJAVADEBUG) {
System.out.println();
System.out.println("SSJAVA: Constructing a flow graph: " + md);
} else {
setPossibleCallees.addAll(ssjava.getCallGraph().getMethods(md));
}
+
+ Set<MethodDescriptor> calleeSet=ssjava.getCallGraph().getCalleeSet(md);
+
+ for (Iterator iterator = calleeSet.iterator(); iterator.hasNext();) {
+ MethodDescriptor calleemd = (MethodDescriptor) iterator.next();
+ if((!ssjava.isSSJavaUtil(calleemd.getClassDesc())) && (! visited.contains(calleemd))){
+ toanalyzeMethodList.add(calleemd);
+ }
+ }
+
mapMethodDescToPossibleMethodDescSet.put(md, setPossibleCallees);
// creates a mapping from a parameter descriptor to its index
FlowGraph fg = new FlowGraph(md, mapParamDescToIdx);
mapMethodDescriptorToFlowGraph.put(md, fg);
+ visited.add(md);
analyzeMethodBody(cd, md);
+
}
}
- }
+// }
_debug_printGraph();
}
ExpressionNode returnExp = rn.getReturnExpression();
- NodeTupleSet nodeSet = new NodeTupleSet();
- analyzeFlowExpressionNode(md, nametable, returnExp, nodeSet, false);
+ if (returnExp != null) {
+ NodeTupleSet nodeSet = new NodeTupleSet();
+ analyzeFlowExpressionNode(md, nametable, returnExp, nodeSet, false);
- FlowGraph fg = getFlowGraph(md);
+ FlowGraph fg = getFlowGraph(md);
- // annotate the elements of the node set as the return location
- for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
- NTuple<Descriptor> returnDescTuple = (NTuple<Descriptor>) iterator.next();
- fg.setReturnFlowNode(returnDescTuple);
- for (Iterator iterator2 = implicitFlowTupleSet.iterator(); iterator2.hasNext();) {
- NTuple<Descriptor> implicitFlowDescTuple = (NTuple<Descriptor>) iterator2.next();
- fg.addValueFlowEdge(implicitFlowDescTuple, returnDescTuple);
+ // annotate the elements of the node set as the return location
+ for (Iterator iterator = nodeSet.iterator(); iterator.hasNext();) {
+ NTuple<Descriptor> returnDescTuple = (NTuple<Descriptor>) iterator.next();
+ fg.setReturnFlowNode(returnDescTuple);
+ for (Iterator iterator2 = implicitFlowTupleSet.iterator(); iterator2.hasNext();) {
+ NTuple<Descriptor> implicitFlowDescTuple = (NTuple<Descriptor>) iterator2.next();
+ fg.addValueFlowEdge(implicitFlowDescTuple, returnDescTuple);
+ }
}
}
} else {
// check 'for loop' case
BlockNode bn = ln.getInitializer();
- analyzeFlowBlockNode(md, bn.getVarTable(), bn, implicitFlowTupleSet);
bn.getVarTable().setParent(nametable);
+ for (int i = 0; i < bn.size(); i++) {
+ BlockStatementNode bsn = bn.get(i);
+ analyzeBlockStatementNode(md, bn.getVarTable(), bsn, implicitFlowTupleSet);
+ }
NodeTupleSet condTupleNode = new NodeTupleSet();
analyzeFlowExpressionNode(md, bn.getVarTable(), ln.getCondition(), condTupleNode, null,
if (min.getExpression() != null) {
NodeTupleSet baseNodeSet = new NodeTupleSet();
- analyzeFlowExpressionNode(calleeMD, nametable, min.getExpression(), baseNodeSet, null,
+ analyzeFlowExpressionNode(md, nametable, min.getExpression(), baseNodeSet, null,
implicitFlowTupleSet, false);
} else {
// checkCallerArgumentLocationConstraints(md, nametable, min,
// baseLocation, constraint);
- if (!min.getMethod().getReturnType().isVoid()) {
+ if (min.getMethod().getReturnType()!=null && !min.getMethod().getReturnType().isVoid()) {
// If method has a return value, compute the highest possible return
// location in the caller's perspective
// CompositeLocation ceilingLoc =
NTuple<Descriptor> argTuple =
analyzeFlowExpressionNode(callermd, nametable, en, new NodeTupleSet(), false);
+ // if argument is liternal node, argTuple is set to NULL.
addArgIdxMap(min, i + offset, argTuple);
}
}
private void analyzeLiteralNode(MethodDescriptor md, SymbolTable nametable, LiteralNode en) {
- // TODO Auto-generated method stub
}
FieldDescriptor fd = (FieldDescriptor) d;
if (fd.isStatic()) {
if (fd.isFinal()) {
- // if it is 'static final', the location has TOP since no one can
- // change its value
- // loc.addLocation(Location.createTopLocation(md));
- // return loc;
+ // if it is 'static final', assign the default TOP LOCATION
+ // DESCRIPTOR
+ base.add(TOPDESC);
+ return base;
} else {
- // if 'static', the location has pre-assigned global loc
- // MethodLattice<String> localLattice = ssjava.getMethodLattice(md);
- // String globalLocId = localLattice.getGlobalLoc();
- // if (globalLocId == null) {
- // throw new
- // Error("Global location element is not defined in the method " +
- // md);
- // }
- // Location globalLoc = new Location(md, globalLocId);
- //
- // loc.addLocation(globalLoc);
+ // if 'static', assign the default GLOBAL LOCATION to the first
+ // element of the tuple
+ base.add(GLOBALDESC);
}
} else {
// the location of field access starts from this, followed by field
base.add(fd);
} else if (d == null) {
// access static field
+ base.add(GLOBALDESC);
+ // base.add(nn.getField());
+ return base;
+
// FieldDescriptor fd = nn.getField();addFlowGraphEdge
//
// MethodLattice<String> localLattice = ssjava.getMethodLattice(md);
}
}
- // if (left instanceof ArrayAccessNode) {
- // ArrayAccessNode aan = (ArrayAccessNode) left;
- // left = aan.getExpression();
- // }
+ if (left instanceof ArrayAccessNode) {
+ ArrayAccessNode aan = (ArrayAccessNode) left;
+ left = aan.getExpression();
+ }
// fanNodeSet
base =
analyzeFlowExpressionNode(md, nametable, left, nodeSet, base, implicitFlowTupleSet, false);
if (!left.getType().isPrimitive()) {
if (fd.getSymbol().equals("length")) {
- // TODO
- // array.length access, return the location of the array
- // return loc;
+ // array.length access, just have the location of the array
+ } else {
+ base.add(fd);
}
- base.add(fd);
}
getFlowGraph(md).createNewFlowNode(base);
}
+ private void debug_printTreeNode(TreeNode tn) {
+
+ System.out.println("DEBUG: " + tn.printNode(0) + " line#=" + tn.getNumLine());
+
+ }
+
private void analyzeFlowAssignmentNode(MethodDescriptor md, SymbolTable nametable,
AssignmentNode an, NTuple<Descriptor> base, NodeTupleSet implicitFlowTupleSet) {
.getBaseOp().getOp() != Operation.POSTDEC)) {
postinc = false;
}
-
// if LHS is array access node, need to capture value flows between an array
// and its index value
analyzeFlowExpressionNode(md, nametable, an.getDest(), nodeSetLHS, null, implicitFlowTupleSet,
analyzeFlowExpressionNode(md, nametable, an.getSrc(), nodeSetRHS, null, implicitFlowTupleSet,
false);
+ if (an.getOperation().getOp() >= 2 && an.getOperation().getOp() <= 12) {
+ // if assignment contains OP+EQ operator, creates edges from LHS to LHS
+ for (Iterator<NTuple<Descriptor>> iter = nodeSetLHS.iterator(); iter.hasNext();) {
+ NTuple<Descriptor> fromTuple = iter.next();
+ for (Iterator<NTuple<Descriptor>> iter2 = nodeSetLHS.iterator(); iter2.hasNext();) {
+ NTuple<Descriptor> toTuple = iter2.next();
+ addFlowGraphEdge(md, fromTuple, toTuple);
+ }
+ }
+ }
+
// creates edges from RHS to LHS
for (Iterator<NTuple<Descriptor>> iter = nodeSetRHS.iterator(); iter.hasNext();) {
NTuple<Descriptor> fromTuple = iter.next();
projects / IRC.git / blobdiff