/*
 * 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 org.apache.bcel.classfile.Code;

import edu.umd.cs.findbugs.BugInstance;
import edu.umd.cs.findbugs.BugReporter;
import edu.umd.cs.findbugs.OpcodeStack;
import edu.umd.cs.findbugs.ba.XMethod;
import edu.umd.cs.findbugs.bcel.OpcodeStackDetector;

/**
 * if we get from a ConcurrentHashMap and assign to a variable... and don't do
 * anything else and perform a null check on it... and then do a set on it...
 * (or anything else inside the if that modifies it?) then we have a bug.
 * 
 * @author Michael Midgley-Biggs
 */
public class AtomicityProblem extends OpcodeStackDetector {

	int priority = IGNORE_PRIORITY;

	int lastQuestionableCheckTarget = -1;

	private BugReporter bugReporter;

	final static boolean DEBUG = false;

	public AtomicityProblem(BugReporter bugReporter) {
		this.bugReporter = bugReporter;
	}

	@Override
	public void visit(Code obj) {
		if (DEBUG) {
			System.out.println("Checking " + obj);
		}
		lastQuestionableCheckTarget = -1;
		super.visit(obj);
	}

	/**
	 * This is the "dumb" version of the detector. It may generate false
	 * positives, and/or not detect all instances of the bug.
	 * 
	 * @see edu.umd.cs.findbugs.visitclass.DismantleBytecode#sawOpcode(int)
	 */
	@Override
	public void sawOpcode(int seen) {
		if (DEBUG) {
			System.out.println(getPC() + " " + OPCODE_NAMES[seen]);
		}
		switch (seen) {
		case IFNE:
		case IFEQ: {
			OpcodeStack.Item top = stack.getStackItem(0);
			if (DEBUG) {
				System.out.println("Stack top: " + top);
			}
			XMethod m = top.getReturnValueOf();
			if (m != null && m.getClassName().equals("java.util.concurrent.ConcurrentHashMap")
			        && m.getName().equals("containsKey")) {
				lastQuestionableCheckTarget = getBranchTarget();
				if (seen == IFEQ) {
					priority = LOW_PRIORITY;
				} else if (seen == IFNE) {
					priority = NORMAL_PRIORITY;
				}
			}
			break;
		}
		case IFNULL:
		case IFNONNULL: {
			OpcodeStack.Item top = stack.getStackItem(0);
			if (DEBUG) {
				System.out.println("Stack top: " + top);
			}
			XMethod m = top.getReturnValueOf();
			if (DEBUG) {
				System.out.println("Found null check");
			}
			if (m != null && m.getClassName().equals("java.util.concurrent.ConcurrentHashMap") && m.getName().equals("get")) {
				lastQuestionableCheckTarget = getBranchTarget();
				if (seen == IFNULL) {
					priority = LOW_PRIORITY;
				} else if (seen == IFNONNULL) {
					priority = NORMAL_PRIORITY;
				}
			}
			break;
		}
		case INVOKEVIRTUAL:
		case INVOKEINTERFACE: {
			if (getDottedClassConstantOperand().equals("java.util.concurrent.ConcurrentHashMap")) {
				String methodName = getNameConstantOperand();
				if (methodName.equals("put")) {
					if ((getPC() < lastQuestionableCheckTarget) && (lastQuestionableCheckTarget != -1)) {
						bugReporter.reportBug(new BugInstance(this, "AT_OPERATION_SEQUENCE_ON_CONCURRENT_ABSTRACTION", priority)
						        .addClassAndMethod(this).addType(getXClassOperand().getClassDescriptor()).addCalledMethod(this)
						        .addSourceLine(this));
					}
				}
			}
			break;
		}
		}
	}
}

@@ -1027,16 +1027,54 @@
 eStack;%0A
+import edu.umd.cs.findbugs.ba.XClass;%0A
 import e
@@ -3537,20 +3537,78 @@
 ();%0A%09%09%09%09
-if (
+XClass xClass = getXClassOperand();%0A%09%09%09%09if (xClass != null && 
 methodNa
@@ -3876,34 +3876,22 @@
 addType(
-getX
+x
 Class
-Operand()
 .getClas

fix NPE warning