/*
* FindBugs - Find bugs in Java programs
* Copyright (C) 2006 University of Maryland
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 2.1 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, write to the Free Software
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
*/
package edu.umd.cs.findbugs.detect;
import java.util.HashMap;
import java.util.Map;
import javax.annotation.CheckForNull;
import org.apache.bcel.classfile.Code;
import edu.umd.cs.findbugs.BugAccumulator;
import edu.umd.cs.findbugs.BugInstance;
import edu.umd.cs.findbugs.BugReporter;
import edu.umd.cs.findbugs.ba.ClassContext;
import edu.umd.cs.findbugs.ba.XFactory;
import edu.umd.cs.findbugs.ba.XMethod;
import edu.umd.cs.findbugs.bcel.OpcodeStackDetector;
import edu.umd.cs.findbugs.internalAnnotations.SlashedClassName;
import edu.umd.cs.findbugs.util.ClassName;
/**
* Detector to find calls to Number constructors with base type argument in Java
* 5 or newer bytecode.
*
* Using <code>new Integer(int)</code> is guaranteed to always result in a new
* object whereas <code>Integer.valueOf(int)</code> allows caching of values to
* be done by the javac, JVM class library or JIT.
*
* Currently only the JVM class library seems to do caching in the range of -128
* to 127. There does not seem to be any caching for float and double which is
* why those are reported as low priority.
*
* All invokes of Number constructor with a constant argument are flagged as
* high priority and invokes with unknwon value are normal priority.
*
* @author Mikko Tiihonen
*/
public class NumberConstructor extends OpcodeStackDetector {
static class Pair {
final XMethod boxingMethod;
public Pair(XMethod boxingMethod, XMethod parsingMethod) {
this.boxingMethod = boxingMethod;
this.parsingMethod = parsingMethod;
}
final XMethod parsingMethod;
}
private final Map<String, Pair> boxClasses = new HashMap<String, Pair>();
private final BugAccumulator bugAccumulator;
/**
* Constructs a NC detector given the reporter to report bugs on
*
* @param bugReporter
* the sync of bug reports
*/
public NumberConstructor(BugReporter bugReporter) {
this.bugAccumulator = new BugAccumulator(bugReporter);
handle("java/lang/Byte", false, "(B)");
handle("java/lang/Character", false, "(C)");
handle("java/lang/Short", false, "(S)");
handle("java/lang/Integer", false, "(I)");
handle("java/lang/Long", false, "(J)");
handle("java/lang/Float", true, "(F)");
handle("java/lang/Double", true, "(D)");
}
private void handle(@SlashedClassName String className, boolean isFloatingPoint, String sig) {
XMethod boxingMethod = XFactory.createXMethod(ClassName.toDottedClassName(className), "valueOf", sig + "L" + className +";", true);
XMethod parsingMethod = XFactory.createXMethod(ClassName.toDottedClassName(className), "valueOf", "(Ljava/lang/String;)" + "L" + className +";", true);
boxClasses.put(className, new Pair(boxingMethod, parsingMethod));
}
/**
* The detector is only meaningful for Java5 class libraries.
*
* @param classContext
* the context object that holds the JavaClass parsed
*/
@Override
public void visitClassContext(ClassContext classContext) {
int majorVersion = classContext.getJavaClass().getMajor();
if (majorVersion >= MAJOR_1_5) {
super.visitClassContext(classContext);
}
}
@Override
public void visit(Code obj) {
super.visit(obj);
bugAccumulator.reportAccumulatedBugs();
}
private boolean matchArguments(String sig1, String sig2) {
int lastParen = sig1.indexOf(')');
String args = sig1.substring(0, lastParen+1);
return sig2.startsWith(args);
}
private @CheckForNull XMethod getShouldCall() {
String cls = getClassConstantOperand();
Pair pair = boxClasses.get(cls);
if (pair == null)
return null;
XMethod shouldCall ;
if (getSigConstantOperand().startsWith("(Ljava/lang/String;)"))
shouldCall = pair.parsingMethod;
else
shouldCall = pair.boxingMethod;
if (shouldCall == null) {
return null;
}
if (matchArguments(getSigConstantOperand(), shouldCall.getSignature()))
return shouldCall;
return null;
}
@Override
public void sawOpcode(int seen) {
// only acts on constructor invoke
if (seen != INVOKESPECIAL) {
return;
}
if (!"<init>".equals(getNameConstantOperand())) {
return;
}
@SlashedClassName String cls = getClassConstantOperand();
XMethod shouldCall = getShouldCall();
if (shouldCall == null)
return;
int prio;
String type;
if (cls.equals("java/lang/Float") || cls.equals("java/lang/Double")) {
prio = LOW_PRIORITY;
type = "DM_FP_NUMBER_CTOR";
} else {
prio = NORMAL_PRIORITY;
Object constantValue = stack.getStackItem(0).getConstant();
if (constantValue instanceof Number) {
long value = ((Number) constantValue).longValue();
if (value < -128 || value > 127)
prio = LOW_PRIORITY;
}
type = "DM_NUMBER_CTOR";
}
BugInstance bug = new BugInstance(this, type, prio).addClass(this).addMethod(this).addCalledMethod(this)
.addMethod(shouldCall).describe("SHOULD_CALL");
bugAccumulator.accumulateBug(bug, this);
}
}