commit 8cdd862dc444d8390d8b6021aaff83efea882e4c
author Pierre De Rop <pderop@apache.org> Sat Apr 24 10:31:41 2010 +0000
committer Pierre De Rop <pderop@apache.org> Sat Apr 24 10:31:41 2010 +0000
tree f9ff9392009043b9d47ae38053d2f9b60bde3ec4
parent af3192d7f9c66960902766f69367cbb76869f11a

added missing synchronize in ManagedServiceFactory's updated method (fix in progress)

git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@937616 13f79535-47bb-0310-9956-ffa450edef68

dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java

diff --git a/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java b/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java
index 40fe7cd..16f23d6 100644
--- a/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java
+++ b/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/AbstractDecorator.java
@@ -48,7 +48,10 @@
     // 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) {