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;
}