You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by dw...@apache.org on 2009/01/07 23:21:45 UTC

svn commit: r732538 - /geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanInstanceState.java

Author: dwoods
Date: Wed Jan  7 14:21:45 2009
New Revision: 732538

URL: http://svn.apache.org/viewvc?rev=732538&view=rev
Log:
GERONIMO-4495 Invoking stop method on a gbean whose instancestate is destoryed caused that gbean in the stopping state.  Patch provided by Ivan (xuhaihong).

Modified:
    geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanInstanceState.java

Modified: geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanInstanceState.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanInstanceState.java?rev=732538&r1=732537&r2=732538&view=diff
==============================================================================
--- geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanInstanceState.java (original)
+++ geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/runtime/GBeanInstanceState.java Wed Jan  7 14:21:45 2009
@@ -152,7 +152,7 @@
         State originalState;
         synchronized (this) {
             originalState = getStateInstance();
-            if (originalState == State.STOPPED) {
+            if (originalState == State.STOPPED || originalState == State.FAILED) {
                 return;
             }