Enable variable-names in Synth. Methods
Using a SummarizedMethodWithNames instead of a normal one enables human readable variable names in WALA-Synthetic methods. This should help when debugging.
This commit is contained in:
parent
ece98a0628
commit
f8970f92b7
|
@ -0,0 +1,166 @@
|
|||
/*
|
||||
* Copyright (c) 2013,
|
||||
* Tobias Blaschke <code@tobiasblaschke.de>
|
||||
* All rights reserved.
|
||||
|
||||
* Redistribution and use in source and binary forms, with or without
|
||||
* modification, are permitted provided that the following conditions are met:
|
||||
*
|
||||
* 1. Redistributions of source code must retain the above copyright notice,
|
||||
* this list of conditions and the following disclaimer.
|
||||
*
|
||||
* 2. Redistributions in binary form must reproduce the above copyright notice,
|
||||
* this list of conditions and the following disclaimer in the documentation
|
||||
* and/or other materials provided with the distribution.
|
||||
*
|
||||
* 3. The names of the contributors may not be used to endorse or promote
|
||||
* products derived from this software without specific prior written
|
||||
* permission.
|
||||
*
|
||||
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
|
||||
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
|
||||
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
|
||||
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
|
||||
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
|
||||
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
|
||||
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
|
||||
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
|
||||
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
|
||||
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
|
||||
* POSSIBILITY OF SUCH DAMAGE.
|
||||
*/
|
||||
package com.ibm.wala.ipa.summaries;
|
||||
|
||||
import com.ibm.wala.ipa.summaries.SummarizedMethod;
|
||||
import com.ibm.wala.ipa.summaries.MethodSummary;
|
||||
import com.ibm.wala.ipa.summaries.SyntheticIR;
|
||||
import com.ibm.wala.cfg.AbstractCFG;
|
||||
|
||||
import com.ibm.wala.classLoader.IClass;
|
||||
import com.ibm.wala.classLoader.SyntheticMethod;
|
||||
import com.ibm.wala.classLoader.IMethod;
|
||||
import com.ibm.wala.ipa.callgraph.Context;
|
||||
import com.ibm.wala.ipa.callgraph.impl.Everywhere;
|
||||
import com.ibm.wala.ssa.IR;
|
||||
import com.ibm.wala.ssa.IR.SSA2LocalMap;
|
||||
import com.ibm.wala.ssa.SSAInstruction;
|
||||
import com.ibm.wala.ssa.ConstantValue;
|
||||
import com.ibm.wala.ssa.SSAOptions;
|
||||
import com.ibm.wala.types.MethodReference;
|
||||
import com.ibm.wala.types.TypeReference;
|
||||
|
||||
import java.util.Map;
|
||||
import com.ibm.wala.util.strings.Atom;
|
||||
|
||||
import org.slf4j.Logger;
|
||||
import org.slf4j.LoggerFactory;
|
||||
|
||||
/**
|
||||
* A SummarizedMethod (for synthetic functions) with variable names.
|
||||
*
|
||||
* Using this class instead of a normal SummarizedMethod enables the use of human-readable variable
|
||||
* names in synthetic methods. This should not change th analysis-result but may come in handy when
|
||||
* debugging.
|
||||
*
|
||||
* @author Tobias Blaschke <code@tobiasblaschke.de>
|
||||
* @since 2013-11-25
|
||||
*/
|
||||
public class SummarizedMethodWithNames extends SummarizedMethod {
|
||||
private static final Logger logger = LoggerFactory.getLogger(SummarizedMethodWithNames.class);
|
||||
|
||||
private final MethodSummary summary;
|
||||
private final Map<Integer, Atom> localNames;
|
||||
|
||||
public SummarizedMethodWithNames(MethodReference ref, MethodSummary summary, IClass declaringClass, Map<Integer, Atom> localNames)
|
||||
throws NullPointerException {
|
||||
super(ref, summary, declaringClass);
|
||||
|
||||
this.summary = summary;
|
||||
this.localNames = localNames;
|
||||
|
||||
logger.debug("From old MSUM");
|
||||
}
|
||||
|
||||
public SummarizedMethodWithNames(MethodReference ref, VolatileMethodSummary summary, IClass declaringClass)
|
||||
throws NullPointerException {
|
||||
super(ref, summary.getMethodSummary(), declaringClass);
|
||||
this.summary = summary.getMethodSummary();
|
||||
this.localNames = ((VolatileMethodSummary)summary).getLocalNames();
|
||||
if (this.localNames.isEmpty()) {
|
||||
logger.warn("Local names are empty");
|
||||
}
|
||||
|
||||
/*{ // DEBUG
|
||||
for (final Integer key : this.localNames.keySet()) {
|
||||
logger.debug("Local " + key + " is " + this.localNames.get(key));
|
||||
}
|
||||
} // */
|
||||
}
|
||||
|
||||
public static class SyntheticIRWithNames extends SyntheticIR {
|
||||
private final SSA2LocalMap localMap;
|
||||
|
||||
public static class SyntheticSSA2LocalMap implements SSA2LocalMap {
|
||||
private final Map<Integer, Atom> localNames;
|
||||
|
||||
public SyntheticSSA2LocalMap(Map<Integer, Atom> localNames) {
|
||||
this.localNames = localNames;
|
||||
}
|
||||
|
||||
/**
|
||||
* Does not respect index.
|
||||
*/
|
||||
@Override
|
||||
public String[] getLocalNames(int index, int vn) {
|
||||
logger.debug("IR.getLocalNames({}, {})", index, vn);
|
||||
if (this.localNames.containsKey(vn)) {
|
||||
return new String[] { this.localNames.get(vn).toString() };
|
||||
} else {
|
||||
return null;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
public SyntheticIRWithNames(IMethod method, Context context, AbstractCFG cfg, SSAInstruction[] instructions,
|
||||
SSAOptions options, Map< Integer, ConstantValue > constants, Map<Integer, Atom> localNames) throws AssertionError {
|
||||
super(method, context, cfg, instructions, options, constants);
|
||||
this.localMap = new SyntheticSSA2LocalMap(localNames);
|
||||
}
|
||||
|
||||
@Override
|
||||
public SSA2LocalMap getLocalMap() {
|
||||
return this.localMap;
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the variable name to a ssa-number.
|
||||
*
|
||||
* Does not respect the value of bcIndex.
|
||||
*/
|
||||
@Override
|
||||
public String getLocalVariableName(int bcIndex, int localNumber) {
|
||||
if (this.localNames.containsKey(localNumber)) {
|
||||
String name = this.localNames.get(localNumber).toString();
|
||||
logger.debug("getLocalVariableName(bc={}, no={}) = {}", bcIndex, localNumber, name);
|
||||
return name;
|
||||
} else {
|
||||
logger.debug("No name for {}", localNumber);
|
||||
return super.getLocalVariableName(bcIndex, localNumber);
|
||||
}
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean hasLocalVariableTable() {
|
||||
return true;
|
||||
}
|
||||
|
||||
@Override
|
||||
public IR makeIR(Context context, SSAOptions options) {
|
||||
final SSAInstruction instrs[] = getStatements(options);
|
||||
final IR ir = new SyntheticIRWithNames(this, Everywhere.EVERYWHERE, makeControlFlowGraph(instrs), instrs,
|
||||
options, summary.getConstants(), localNames);
|
||||
|
||||
return ir;
|
||||
}
|
||||
}
|
|
@ -51,6 +51,7 @@ import static org.jf.dexlib.Util.AccessFlags.VOLATILE;
|
|||
|
||||
import java.util.ArrayList;
|
||||
import java.util.Collection;
|
||||
import java.util.Collections;
|
||||
|
||||
import org.jf.dexlib.AnnotationItem;
|
||||
import org.jf.dexlib.AnnotationSetItem;
|
||||
|
@ -208,8 +209,13 @@ public class DexIMethod implements IBytecodeMethod {
|
|||
public TypeReference[] getDeclaredExceptions()
|
||||
throws InvalidClassFileException, UnsupportedOperationException {
|
||||
|
||||
ArrayList<String> strings = new ArrayList<String>();
|
||||
AnnotationSetItem annotationSet = myClass.getClassDefItem().getAnnotations().getMethodAnnotations(eMethod.method);
|
||||
assert (eMethod.method != null);
|
||||
if (myClass.getClassDefItem().getAnnotations() == null) {
|
||||
logger.error("Get Annotations is null for method " + eMethod + " in getDeclaredExceptions");
|
||||
return null;
|
||||
}
|
||||
ArrayList<String> strings = new ArrayList<String>();
|
||||
AnnotationSetItem annotationSet = myClass.getClassDefItem().getAnnotations().getMethodAnnotations(eMethod.method);
|
||||
if (annotationSet != null) {
|
||||
for (AnnotationItem annotationItem: annotationSet.getAnnotations())
|
||||
{
|
||||
|
@ -3283,11 +3289,48 @@ public class DexIMethod implements IBytecodeMethod {
|
|||
// }
|
||||
// }
|
||||
|
||||
/**
|
||||
*
|
||||
* hrows InvalidClassFileException
|
||||
* @throws UnsupportedOperationException
|
||||
*
|
||||
* @todo Review this implementation - it may be horribly wrong!
|
||||
*/
|
||||
@Override
|
||||
public Collection<CallSiteReference> getCallSites()
|
||||
throws InvalidClassFileException {
|
||||
throw new UnsupportedOperationException("getCallSites() not implemented");
|
||||
//return null;
|
||||
Collection<CallSiteReference> empty = Collections.emptySet();
|
||||
if (isNative()) {
|
||||
return empty;
|
||||
}
|
||||
|
||||
assert(false) : "Please review getCallSites-Implementation before use!"; // TODO
|
||||
|
||||
ArrayList<CallSiteReference> csites = new ArrayList<CallSiteReference>();
|
||||
// XXX The call Sites in this method or to this method?!!!
|
||||
for (Instruction inst: instructions()) {
|
||||
if (inst instanceof Invoke) {
|
||||
// Locate the Target
|
||||
MethodReference target;
|
||||
ClassLoaderReference loader = ClassLoaderReference.Primordial;
|
||||
target = MethodReference.findOrCreate(
|
||||
loader, // XXX: Is this the correct class loader?
|
||||
((Invoke)inst).clazzName,
|
||||
((Invoke)inst).methodName,
|
||||
((Invoke)inst).descriptor );
|
||||
|
||||
csites.add(
|
||||
CallSiteReference.make(
|
||||
inst.pc, // programCounter
|
||||
target, // declaredTarget
|
||||
((Invoke)inst).getInvocationCode() // invocationCode
|
||||
));
|
||||
logger.info("\tClass Name:\t" + ((Invoke)inst).clazzName);
|
||||
logger.info("\tMethod Name:\t" + ((Invoke)inst).methodName);
|
||||
logger.info("\tSignature:\t" + ((Invoke)inst).descriptor );
|
||||
}
|
||||
}
|
||||
return Collections.unmodifiableCollection(csites);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
|
Loading…
Reference in New Issue