You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by rg...@apache.org on 2014/07/21 16:43:37 UTC

svn commit: r1612284 - /qpid/trunk/qpid/java/amqp-1-0-common/src/main/java/org/apache/qpid/amqp_1_0/transport/ConnectionEndpoint.java

Author: rgodfrey
Date: Mon Jul 21 14:43:36 2014
New Revision: 1612284

URL: http://svn.apache.org/r1612284
Log:
QPID-5884 : The broker should not NPE if it receives a SASL Init request where no mechanism is provided

Modified:
    qpid/trunk/qpid/java/amqp-1-0-common/src/main/java/org/apache/qpid/amqp_1_0/transport/ConnectionEndpoint.java

Modified: qpid/trunk/qpid/java/amqp-1-0-common/src/main/java/org/apache/qpid/amqp_1_0/transport/ConnectionEndpoint.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/amqp-1-0-common/src/main/java/org/apache/qpid/amqp_1_0/transport/ConnectionEndpoint.java?rev=1612284&r1=1612283&r2=1612284&view=diff
==============================================================================
--- qpid/trunk/qpid/java/amqp-1-0-common/src/main/java/org/apache/qpid/amqp_1_0/transport/ConnectionEndpoint.java (original)
+++ qpid/trunk/qpid/java/amqp-1-0-common/src/main/java/org/apache/qpid/amqp_1_0/transport/ConnectionEndpoint.java Mon Jul 21 14:43:36 2014
@@ -851,14 +851,14 @@ public class ConnectionEndpoint implemen
 
     public void receiveSaslInit(final SaslInit saslInit)
     {
-        Symbol mechanism = saslInit.getMechanism();
+        String mechanism = saslInit.getMechanism() == null ? null : saslInit.getMechanism().toString();
         final Binary initialResponse = saslInit.getInitialResponse();
         byte[] response = initialResponse == null ? new byte[0] : initialResponse.getArray();
 
 
         try
         {
-            _saslServer = _saslServerProvider.getSaslServer(mechanism.toString(), "localhost");
+            _saslServer = _saslServerProvider.getSaslServer(mechanism, "localhost");
 
             // Process response from the client
             byte[] challenge = _saslServer.evaluateResponse(response != null ? response : new byte[0]);



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org