You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by gn...@apache.org on 2013/08/01 13:40:02 UTC

svn commit: r1509177 - /karaf/branches/karaf-2.3.x/admin/core/src/main/java/org/apache/karaf/admin/internal/AdminServiceImpl.java

Author: gnodet
Date: Thu Aug  1 11:40:02 2013
New Revision: 1509177

URL: http://svn.apache.org/r1509177
Log:
[KARAF-2423] Avoid overlapping lock exceptions when multiple threads access the admin service

Modified:
    karaf/branches/karaf-2.3.x/admin/core/src/main/java/org/apache/karaf/admin/internal/AdminServiceImpl.java

Modified: karaf/branches/karaf-2.3.x/admin/core/src/main/java/org/apache/karaf/admin/internal/AdminServiceImpl.java
URL: http://svn.apache.org/viewvc/karaf/branches/karaf-2.3.x/admin/core/src/main/java/org/apache/karaf/admin/internal/AdminServiceImpl.java?rev=1509177&r1=1509176&r2=1509177&view=diff
==============================================================================
--- karaf/branches/karaf-2.3.x/admin/core/src/main/java/org/apache/karaf/admin/internal/AdminServiceImpl.java (original)
+++ karaf/branches/karaf-2.3.x/admin/core/src/main/java/org/apache/karaf/admin/internal/AdminServiceImpl.java Thu Aug  1 11:40:02 2013
@@ -197,7 +197,7 @@ public class AdminServiceImpl implements
         T call(State state) throws IOException;
     }
 
-    <T> T execute(final Task<T> callback) {
+    synchronized <T> T execute(final Task<T> callback) {
         final File storageFile = new File(storageLocation, STORAGE_FILE);
         if (!storageFile.exists()) {
             storageFile.getParentFile().mkdirs();