Repository: activemq
Updated Branches:
  refs/heads/master 386210faf -> 90726a60a


https://issues.apache.org/jira/browse/AMQ-6153

Only setting the socketBufferSize in TcpTransport if the value is
greater than 0 as other values are illegal.


Project: http://git-wip-us.apache.org/repos/asf/activemq/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq/commit/90726a60
Tree: http://git-wip-us.apache.org/repos/asf/activemq/tree/90726a60
Diff: http://git-wip-us.apache.org/repos/asf/activemq/diff/90726a60

Branch: refs/heads/master
Commit: 90726a60afffa0bff6db74b15340152ebe1d5082
Parents: 386210f
Author: Christopher L. Shannon (cshannon) <[email protected]>
Authored: Mon Feb 8 16:11:55 2016 +0000
Committer: Christopher L. Shannon (cshannon) <[email protected]>
Committed: Mon Feb 8 16:13:01 2016 +0000

----------------------------------------------------------------------
 .../org/apache/activemq/transport/tcp/TcpTransport.java     | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq/blob/90726a60/activemq-client/src/main/java/org/apache/activemq/transport/tcp/TcpTransport.java
----------------------------------------------------------------------
diff --git 
a/activemq-client/src/main/java/org/apache/activemq/transport/tcp/TcpTransport.java
 
b/activemq-client/src/main/java/org/apache/activemq/transport/tcp/TcpTransport.java
index 335cde7..60fe283 100755
--- 
a/activemq-client/src/main/java/org/apache/activemq/transport/tcp/TcpTransport.java
+++ 
b/activemq-client/src/main/java/org/apache/activemq/transport/tcp/TcpTransport.java
@@ -453,8 +453,13 @@ public class TcpTransport extends TransportThreadSupport 
implements Transport, S
         }
 
         try {
-            sock.setReceiveBufferSize(socketBufferSize);
-            sock.setSendBufferSize(socketBufferSize);
+            //only positive values are legal
+            if (socketBufferSize > 0) {
+                sock.setReceiveBufferSize(socketBufferSize);
+                sock.setSendBufferSize(socketBufferSize);
+            } else {
+                LOG.warn("Socket buffer size was set to {}; Skipping this 
setting as the size must be a positive number.", socketBufferSize);
+            }
         } catch (SocketException se) {
             LOG.warn("Cannot set socket buffer size = " + socketBufferSize);
             LOG.debug("Cannot set socket buffer size. Reason: " + 
se.getMessage() + ". This exception is ignored.", se);

Reply via email to