[jboss-cvs] JBossRemoting/src/main/org/jboss/remoting ...

[Ron Sigal]

  User: rsigal  
  Date: 06/11/21 02:05:03

  Modified:    src/main/org/jboss/remoting  Tag: remoting_2_x Client.java
  Log:
  JBREM-605:  Added test for consistency of listenerIds in acknowledgeCallbacks().
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.53.2.4  +6 -1      JBossRemoting/src/main/org/jboss/remoting/Client.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Client.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/Client.java,v
  retrieving revision 1.53.2.3
  retrieving revision 1.53.2.4
  diff -u -b -r1.53.2.3 -r1.53.2.4
  --- Client.java	16 Nov 2006 07:18:43 -0000	1.53.2.3
  +++ Client.java	21 Nov 2006 07:05:03 -0000	1.53.2.4
  @@ -63,7 +63,7 @@
    *
    * @author <a href="mailto:jhaynie at vocalocity.net">Jeff Haynie</a>
    * @author <a href="mailto:telrod at e2technologies.net">Tom Elrod</a>
  - * @version $Revision: 1.53.2.3 $
  + * @version $Revision: 1.53.2.4 $
    */
   public class Client implements Externalizable
   {
  @@ -1271,6 +1271,11 @@
                        responseList.add(response);
                     
                     String nextListenerId = (String) returnPayload.get(LISTENER_ID_KEY);
  +                  if (nextListenerId == null)
  +                  {
  +                     throw new Exception("Cannot acknowledge callbacks: " +
  +                                         "callback " + callbackId + " has null listener id");
  +                  }
                     if (i == 0)
                     {
                        listenerId = nextListenerId;