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

[Ron Sigal]

  User: rsigal  
  Date: 07/01/17 03:48:05

  Modified:    src/main/org/jboss/remoting  Tag: remoting_2_x
                        ConnectionValidator.java
  Log:
  JBREM-662:  (1) checkConnection() makes a copy of config map; (2) addConnectionListener synch's on listeners (instead of listener).
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.13.2.4  +6 -2      JBossRemoting/src/main/org/jboss/remoting/ConnectionValidator.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: ConnectionValidator.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossRemoting/src/main/org/jboss/remoting/ConnectionValidator.java,v
  retrieving revision 1.13.2.3
  retrieving revision 1.13.2.4
  diff -u -b -r1.13.2.3 -r1.13.2.4
  --- ConnectionValidator.java	10 Jan 2007 10:30:33 -0000	1.13.2.3
  +++ ConnectionValidator.java	17 Jan 2007 08:48:05 -0000	1.13.2.4
  @@ -96,7 +96,7 @@
      {
         if (listener != null)
         {
  -         synchronized (listener)
  +         synchronized (listeners)
            {
               if (listeners.size() == 0)
               {
  @@ -191,11 +191,15 @@
      {
         boolean pingWorked = false;
   
  -      Map configMap = config;
  +      Map configMap = null;
         if (configMap == null)
         {
            configMap = new HashMap();
         }
  +      else
  +      {
  +         configMap = new HashMap(config);
  +      }
         configMap.put("connection_checker", "true");
         configMap.put("timeout", "1000");
         configMap.put("NumberOfRetries", "1");