You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2009/06/30 03:24:40 UTC

svn commit: r789528 - /cxf/trunk/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java

Author: dkulp
Date: Tue Jun 30 01:24:39 2009
New Revision: 789528

URL: http://svn.apache.org/viewvc?rev=789528&view=rev
Log:
[CXF-2317] Fix for NPE in RetransmissionQueueImpl - fix from Guillaume
Porcher applied.

Modified:
    cxf/trunk/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java

Modified: cxf/trunk/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java?rev=789528&r1=789527&r2=789528&view=diff
==============================================================================
--- cxf/trunk/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java (original)
+++ cxf/trunk/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java Tue Jun 30 01:24:39 2009
@@ -131,9 +131,9 @@
                         purged.add(m);
                     }
                 }
-            }
-            if (sequenceCandidates.isEmpty()) {
-                candidates.remove(seq.getIdentifier().getValue());
+                if (sequenceCandidates.isEmpty()) {
+                    candidates.remove(seq.getIdentifier().getValue());
+                }
             }
             LOG.fine("Completed purging resend candidates.");
         }