You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by fm...@apache.org on 2009/10/30 14:32:14 UTC

svn commit: r831291 - /felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java

Author: fmeschbe
Date: Fri Oct 30 13:32:14 2009
New Revision: 831291

URL: http://svn.apache.org/viewvc?rev=831291&view=rev
Log:
FELIX-1826 The m_componentInstances map contains entries where the key is
the value (and not the ComponentInstance object), which may lead to an
ArrayStoreException when calling getComponentManagers

Modified:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java?rev=831291&r1=831290&r2=831291&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/ComponentFactoryImpl.java Fri Oct 30 13:32:14 2009
@@ -118,7 +118,7 @@
             throw new ComponentException( "Failed activating component" );
         }
 
-        m_componentInstances.put( cm, instance );
+        m_componentInstances.put( cm, cm );
         return instance;
     }