- Waiting for acceptor thread to close server socket in NioServer bef…

…ore returning (https://issues.redhat.com/browse/JGRP-2647)

- Changed semantics of FD_SOCK2.port_range: 0 means test *1* port. This aligns the semantics with those of other protocols, e.g. FD_SOCK

src/org/jgroups/blocks/cs/NioServer.java

@@ -116,6 +116,8 @@ public synchronized void stop() {
             // Util.close(selector); // closing the selector also stops the acceptor thread
             // socket_factory.close(channel);
             selector.wakeup();
+            // Wait for server channel to close (via acceptorDone())
+            Util.interruptAndWaitToDie(acceptor);
         }
     }
 

src/org/jgroups/protocols/FD_SOCK2.java

@@ -450,8 +450,8 @@ public static ByteArray messageToBuffer(Message msg) throws Exception {
     }
 
     protected int[] computeBindPorts(int actual_port) {
-        int[] bind_ports=new int[port_range];
-        for(int i=0; i < port_range; i++) {
+        int[] bind_ports=new int[port_range+1];
+        for(int i=0; i <= port_range; i++) {
             int port=(actual_port+offset+i) % max_port;
             if(port < min_port)
                 port=port+min_port;