Author: pderop
Date: Sat Apr 24 10:31:41 2010
New Revision: 937616
URL: http://svn.apache.org/viewvc?rev=937616&view=rev
Log:
added missing synchronize in ManagedServiceFactory's updated method (fix in
progress)
Modified:
felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java
Modified:
felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java
URL:
http://svn.apache.org/viewvc/felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java?rev=937616&r1=937615&r2=937616&view=diff
==============================================================================
---
felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java
(original)
+++
felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java
Sat Apr 24 10:31:41 2010
@@ -48,7 +48,10 @@ public abstract class AbstractDecorator
// callbacks for FactoryConfigurationAdapterImpl
public void updated(String pid, Dictionary properties) throws
ConfigurationException {
try {
- Service service = (Service) m_services.get(pid);
+ Service service;
+ synchronized (this) {
+ service = (Service) m_services.get(pid);
+ }
if (service == null) {
service = createService(new Object[] { properties });
synchronized (this) {
