Changeset 5312

Timestamp:

08/23/10 18:07:41 (18 months ago)

Author:

mtld

Message:

sessionstatuslistener race condition fix, fixed webservices-core lib

Location:

branches/release-0.13.5-before-modinc/pfixcore

Files:

• . (modified) (1 prop)
• ChangeLog (modified) (1 diff)
• lib/pustefix-webservices-core-0.1.2.jar (deleted)
• lib/pustefix-webservices-core-0.2.3.jar (added)
• src/de/schlund/pfixcore/workflow/context/SessionContextImpl.java (modified) (1 diff)

branches/release-0.13.5-before-modinc/pfixcore/ChangeLog

@@ +1 @@
+2009-07-27  Martin Leidig  <mleidig@schlund.de>
+
+        Fixed possible deadlock within SessionEndNotificator when
+        one thread adds a SessionListener after getting a lock to an 
+        application object, while another thread is invalidating 
+        the session and calls a registered SessionListener, which itself
+        tries to lock the same application object.
+
 2009-06-25  Martin Leidig  <mleidig@schlund.de>
 

branches/release-0.13.5-before-modinc/pfixcore/src/de/schlund/pfixcore/workflow/context/SessionContextImpl.java

@@ -61 +61 @@
             // Send event to registered listeners
             try {
+                SessionStatusListener[] currentListeners;
                 synchronized (this) {
-                    for (SessionStatusListener l : sessionListeners) {
-                        l.sessionStatusChanged(new SessionStatusEvent(SessionStatusEvent.Type.SESSION_DESTROYED));
-                    }
+                    currentListeners = new SessionStatusListener[sessionListeners.size()];
+                    sessionListeners.toArray(currentListeners);
+                }
+                for (SessionStatusListener l : currentListeners) {
+                    l.sessionStatusChanged(new SessionStatusEvent(SessionStatusEvent.Type.SESSION_DESTROYED));
                 }
             } catch(Throwable t) {