src.main.java.com.mebigfatguy.fbcontrib.detect.NonOwnedSynchronization Maven / Gradle / Ivy
Go to download
Show more of this group Show more artifacts with this name
Show all versions of sb-contrib Show documentation
Show all versions of sb-contrib Show documentation
An auxiliary findbugs.sourceforge.net plugin for java bug detectors that fall outside the narrow scope of detectors to be packaged with the product itself.
The newest version!
/*
* fb-contrib - Auxiliary detectors for Java programs
* Copyright (C) 2005-2019 Dave Brosius
*
* 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 com.mebigfatguy.fbcontrib.detect;
import java.util.BitSet;
import java.util.HashMap;
import java.util.Map;
import org.apache.bcel.Const;
import org.apache.bcel.classfile.Code;
import org.apache.bcel.classfile.Method;
import com.mebigfatguy.fbcontrib.utils.BugType;
import com.mebigfatguy.fbcontrib.utils.RegisterUtils;
import com.mebigfatguy.fbcontrib.utils.SignatureUtils;
import com.mebigfatguy.fbcontrib.utils.TernaryPatcher;
import com.mebigfatguy.fbcontrib.utils.Values;
import edu.umd.cs.findbugs.BugInstance;
import edu.umd.cs.findbugs.BugReporter;
import edu.umd.cs.findbugs.BytecodeScanningDetector;
import edu.umd.cs.findbugs.OpcodeStack;
import edu.umd.cs.findbugs.OpcodeStack.CustomUserValue;
import edu.umd.cs.findbugs.ba.ClassContext;
/**
* looks for methods that synchronize on variables that are not owned by the
* current class. Doing this causes confusion when two classes use the same
* variable for their own synchronization purposes. For cleanest separation of
* interests, only synchronize on private fields of the class. Note that 'this'
* is not owned by the current class and synchronization on 'this' should be
* avoided as well.
*/
@CustomUserValue
public class NonOwnedSynchronization extends BytecodeScanningDetector {
private static final Integer OWNED = Integer.valueOf(Integer.MAX_VALUE);
private final BugReporter bugReporter;
private OpcodeStack stack;
private Map regPriorities;
/**
* constructs a NOS detector given the reporter to report bugs on
*
* @param bugReporter the sync of bug reports
*/
public NonOwnedSynchronization(BugReporter bugReporter) {
this.bugReporter = bugReporter;
}
/**
* implements the visitor to set and clear the stack and priorities
*/
@Override
public void visitClassContext(ClassContext classContext) {
try {
stack = new OpcodeStack();
regPriorities = new HashMap<>();
super.visitClassContext(classContext);
} finally {
stack = null;
regPriorities = null;
}
}
/**
* looks for methods that contain a MONITORENTER opcode
*
* @param method the context object of the current method
* @return if the class uses synchronization
*/
public boolean prescreen(Method method) {
BitSet bytecodeSet = getClassContext().getBytecodeSet(method);
return (bytecodeSet != null) && (bytecodeSet.get(Const.MONITORENTER));
}
/**
* implements the visitor to reset the opcode stack
*
* @param obj the context object of the currently parsed code block
*/
@Override
public void visitCode(Code obj) {
Method method = getMethod();
if (prescreen(method)) {
stack.resetForMethodEntry(this);
regPriorities.clear();
int[] parmRegs = RegisterUtils.getParameterRegisters(method);
for (int reg : parmRegs) {
regPriorities.put(Integer.valueOf(reg), Values.NORMAL_BUG_PRIORITY);
}
if (!method.isStatic()) {
regPriorities.put(Values.ZERO, Values.LOW_BUG_PRIORITY);
}
super.visitCode(obj);
}
}
/**
* implements the visitor to look for synchronization on non-owned objects
*
* @param seen the opcode of the currently parsed instruction
*/
@Override
public void sawOpcode(int seen) {
Integer tosIsPriority = null;
try {
stack.precomputation(this);
switch (seen) {
case Const.GETFIELD:
tosIsPriority = OWNED;
break;
case Const.ALOAD:
case Const.ALOAD_0:
case Const.ALOAD_1:
case Const.ALOAD_2:
case Const.ALOAD_3: {
int reg = RegisterUtils.getALoadReg(this, seen);
if ((reg == 0) && getMethod().isStatic()) {
tosIsPriority = Values.LOW_BUG_PRIORITY;
} else {
tosIsPriority = regPriorities.get(Integer.valueOf(reg));
if (tosIsPriority == null) {
tosIsPriority = Values.NORMAL_BUG_PRIORITY;
}
}
}
break;
case Const.ASTORE:
case Const.ASTORE_0:
case Const.ASTORE_1:
case Const.ASTORE_2:
case Const.ASTORE_3: {
if (stack.getStackDepth() > 0) {
OpcodeStack.Item item = stack.getStackItem(0);
Integer priority = (Integer) item.getUserValue();
regPriorities.put(Integer.valueOf(RegisterUtils.getAStoreReg(this, seen)), priority);
}
}
break;
case Const.INVOKEVIRTUAL:
case Const.INVOKEINTERFACE: {
String sig = getSigConstantOperand();
if (SignatureUtils.getReturnSignature(sig).startsWith(Values.SIG_QUALIFIED_CLASS_PREFIX)) {
int parmCnt = SignatureUtils.getNumParameters(sig);
if (stack.getStackDepth() > parmCnt) {
OpcodeStack.Item itm = stack.getStackItem(parmCnt);
Integer priority = (Integer) itm.getUserValue();
if ((priority != null) && OWNED.equals(priority)) {
tosIsPriority = OWNED;
} else {
int reg = itm.getRegisterNumber();
if (reg > 0) {
tosIsPriority = regPriorities.get(Integer.valueOf(reg));
} else {
tosIsPriority = OWNED;
}
}
}
}
}
break;
case Const.INVOKESTATIC: {
String sig = getSigConstantOperand();
if (SignatureUtils.getReturnSignature(sig).startsWith(Values.SIG_QUALIFIED_CLASS_PREFIX)) {
tosIsPriority = OWNED; // can't be sure
}
}
break;
case Const.INVOKESPECIAL: {
String name = getNameConstantOperand();
if (Values.CONSTRUCTOR.equals(name)) {
tosIsPriority = OWNED;
}
}
break;
case Const.MONITORENTER: {
if (stack.getStackDepth() > 0) {
OpcodeStack.Item itm = stack.getStackItem(0);
Integer priority = (Integer) itm.getUserValue();
if ((priority != null) && (!priority.equals(OWNED))) {
bugReporter.reportBug(
new BugInstance(this, BugType.NOS_NON_OWNED_SYNCHRONIZATION.name(), priority.intValue())
.addClass(this).addMethod(this).addSourceLine(this));
}
}
}
break;
default:
break;
}
} finally {
TernaryPatcher.pre(stack, seen);
stack.sawOpcode(this, seen);
TernaryPatcher.post(stack, seen);
if ((tosIsPriority != null) && (stack.getStackDepth() > 0)) {
OpcodeStack.Item itm = stack.getStackItem(0);
itm.setUserValue(tosIsPriority);
}
}
}
}