edu.umd.cs.findbugs.detect.FindNullDerefsInvolvingNonShortCircuitEvaluation Maven / Gradle / Ivy
Go to download
Show more of this group Show more artifacts with this name
Show all versions of spotbugs Show documentation
Show all versions of spotbugs Show documentation
SpotBugs: Because it's easy!
/*
* FindBugs - Find bugs in Java programs
* Copyright (C) 2004-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.Iterator;
import java.util.Set;
import javax.annotation.CheckForNull;
import org.apache.bcel.Const;
import org.apache.bcel.classfile.Code;
import org.apache.bcel.generic.IfInstruction;
import org.apache.bcel.generic.InstructionHandle;
import edu.umd.cs.findbugs.BugAnnotation;
import edu.umd.cs.findbugs.BugInstance;
import edu.umd.cs.findbugs.BugReporter;
import edu.umd.cs.findbugs.OpcodeStack;
import edu.umd.cs.findbugs.SourceLineAnnotation;
import edu.umd.cs.findbugs.ba.CFG;
import edu.umd.cs.findbugs.ba.CFGBuilderException;
import edu.umd.cs.findbugs.ba.DataflowAnalysisException;
import edu.umd.cs.findbugs.ba.Location;
import edu.umd.cs.findbugs.ba.deref.UnconditionalValueDerefDataflow;
import edu.umd.cs.findbugs.ba.deref.UnconditionalValueDerefSet;
import edu.umd.cs.findbugs.ba.npe.IsNullValueDataflow;
import edu.umd.cs.findbugs.ba.npe.IsNullValueFrame;
import edu.umd.cs.findbugs.ba.vna.ValueNumber;
import edu.umd.cs.findbugs.ba.vna.ValueNumberDataflow;
import edu.umd.cs.findbugs.ba.vna.ValueNumberFrame;
import edu.umd.cs.findbugs.ba.vna.ValueNumberSourceInfo;
import edu.umd.cs.findbugs.bcel.OpcodeStackDetector;
public class FindNullDerefsInvolvingNonShortCircuitEvaluation extends OpcodeStackDetector {
private static boolean DEBUG = false;
BugReporter bugReporter;
public FindNullDerefsInvolvingNonShortCircuitEvaluation(BugReporter bugReporter) {
this.bugReporter = bugReporter;
}
@Override
public void visit(Code code) {
boolean interesting = true;
if (interesting) {
// initialize any variables we want to initialize for the method
super.visit(code); // make callbacks to sawOpcode for all opcodes
}
}
@Override
public void sawOpcode(int seen) {
if (seen == Const.IAND || seen == Const.IOR) {
int nextOpcode = getCodeByte(getPC() + 1);
if (nextOpcode == Const.IFEQ || nextOpcode == Const.IFNE) {
OpcodeStack.Item left = stack.getStackItem(1);
OpcodeStack.Item right = stack.getStackItem(0);
checkForNullForcingABranch(seen, nextOpcode, left);
checkForNullForcingABranch(seen, nextOpcode, right);
}
}
}
private void checkForNullForcingABranch(int seen, int nextOpcode, OpcodeStack.Item item) {
if (nullGuaranteesBranch(seen, item)) {
// null guarantees a branch
boolean nullGuaranteesZero = seen == Const.IAND;
boolean nullGuaranteesBranch = nullGuaranteesZero ^ (nextOpcode == Const.IFNE);
if (DEBUG) {
System.out.println(item.getPC() + " null guarantees " + nullGuaranteesBranch + " branch");
}
try {
CFG cfg = getClassContext().getCFG(getMethod());
Location produced = findLocation(cfg, item.getPC());
Location branch = findLocation(cfg, getPC() + 1);
if (produced == null || branch == null) {
return;
}
IfInstruction branchInstruction = (IfInstruction) branch.getHandle().getInstruction();
IsNullValueDataflow isNullValueDataflow = getClassContext().getIsNullValueDataflow(getMethod());
ValueNumberDataflow valueNumberDataflow = getClassContext().getValueNumberDataflow(getMethod());
UnconditionalValueDerefDataflow unconditionalValueDerefDataflow = getClassContext()
.getUnconditionalValueDerefDataflow(getMethod());
ValueNumberFrame valueNumberFact = valueNumberDataflow.getFactAtLocation(produced);
IsNullValueFrame isNullFact = isNullValueDataflow.getFactAtLocation(produced);
ValueNumber value = valueNumberFact.getTopValue();
if (isNullFact.getTopValue().isDefinitelyNotNull()) {
return;
}
if (DEBUG) {
System.out.println("Produced: " + produced);
System.out.println(valueNumberFact);
System.out.println(isNullFact);
System.out.println("value: " + value);
System.out.println("branch: " + branch);
System.out.println("instruction: " + branchInstruction);
System.out.println("target: " + branchInstruction.getTarget());
System.out.println("next: " + branch.getHandle().getNext());
}
Location guaranteed = findLocation(cfg, nullGuaranteesBranch ? branchInstruction.getTarget()
: branch.getHandle()
.getNext());
if (guaranteed == null) {
return;
}
UnconditionalValueDerefSet unconditionalDeref = unconditionalValueDerefDataflow.getFactAtLocation(guaranteed);
if (DEBUG) {
System.out.println("Guaranteed on null: " + guaranteed);
System.out.println(unconditionalDeref);
}
if (unconditionalDeref.isUnconditionallyDereferenced(value)) {
SourceLineAnnotation tested = SourceLineAnnotation.fromVisitedInstruction(getClassContext(), getMethod(),
produced);
BugAnnotation variableAnnotation = ValueNumberSourceInfo.findAnnotationFromValueNumber(getMethod(), produced,
value, valueNumberFact, "VALUE_OF");
Set unconditionalDerefLocationSet = unconditionalDeref.getUnconditionalDerefLocationSet(value);
BugInstance bug;
if (unconditionalDerefLocationSet.size() > 1) {
bug = new BugInstance(this, "NP_GUARANTEED_DEREF", NORMAL_PRIORITY).addClassAndMethod(this);
bug.addOptionalAnnotation(variableAnnotation);
bug.addSourceLine(tested).describe("SOURCE_LINE_KNOWN_NULL");
for (Location dereferenced : unconditionalDerefLocationSet) {
bug.addSourceLine(getClassContext(), getMethod(), dereferenced).describe("SOURCE_LINE_DEREF");
}
} else {
bug = new BugInstance(this, "NP_NULL_ON_SOME_PATH", NORMAL_PRIORITY).addClassAndMethod(this);
bug.addOptionalAnnotation(variableAnnotation);
for (Location dereferenced : unconditionalDerefLocationSet) {
bug.addSourceLine(getClassContext(), getMethod(), dereferenced).describe("SOURCE_LINE_DEREF");
}
bug.addSourceLine(tested).describe("SOURCE_LINE_KNOWN_NULL");
}
bugReporter.reportBug(bug);
}
} catch (DataflowAnalysisException e) {
bugReporter.logError("Error getting analysis for " + getFullyQualifiedMethodName(), e);
} catch (CFGBuilderException e) {
bugReporter.logError("Error getting analysis for " + getFullyQualifiedMethodName(), e);
}
}
}
@CheckForNull
Location findLocation(CFG cfg, int pc) {
for (Iterator i = cfg.locationIterator(); i.hasNext();) {
Location loc = i.next();
if (loc.getHandle().getPosition() == pc) {
return loc;
}
}
return null;
}
@CheckForNull
Location findLocation(CFG cfg, InstructionHandle handle) {
for (Iterator i = cfg.locationIterator(); i.hasNext();) {
Location loc = i.next();
if (loc.getHandle() == handle) {
return loc;
}
}
return null;
}
private boolean nullGuaranteesBranch(int seen, OpcodeStack.Item item) {
return item.getSpecialKind() == OpcodeStack.Item.ZERO_MEANS_NULL && seen == Const.IAND
|| item.getSpecialKind() == OpcodeStack.Item.NONZERO_MEANS_NULL && seen == Const.IOR;
}
/*
private void emitWarning() {
System.out.println("Warn about " + getMethodName());
}
*/
}