Author: kwall
Date: Fri Jan  8 12:34:06 2016
New Revision: 1723717

URL: http://svn.apache.org/viewvc?rev=1723717&view=rev
Log:
QPID-6939: Merged from trunk

Modified:
    qpid/java/branches/6.0.x/   (props changed)
    
qpid/java/branches/6.0.x/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/AMQPConnection_1_0.java

Propchange: qpid/java/branches/6.0.x/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Jan  8 12:34:06 2016
@@ -9,5 +9,5 @@
 /qpid/branches/java-broker-vhost-refactor/java:1493674-1494547
 /qpid/branches/java-network-refactor/qpid/java:805429-821809
 /qpid/branches/qpid-2935/qpid/java:1061302-1072333
-/qpid/java/trunk:1715445-1715447,1715586,1715940,1716086-1716087,1716127-1716128,1716141,1716153,1716155,1716194,1716204,1716209,1716227,1716277,1716357,1716368,1716370,1716374,1716432,1716444-1716445,1716455,1716461,1716474,1716489,1716497,1716515,1716555,1716602,1716606-1716610,1716619,1716636,1717269,1717299,1717401,1717446,1717449,1717626,1717691,1717735,1717780,1718744,1723064
+/qpid/java/trunk:1715445-1715447,1715586,1715940,1716086-1716087,1716127-1716128,1716141,1716153,1716155,1716194,1716204,1716209,1716227,1716277,1716357,1716368,1716370,1716374,1716432,1716444-1716445,1716455,1716461,1716474,1716489,1716497,1716515,1716555,1716602,1716606-1716610,1716619,1716636,1717269,1717299,1717401,1717446,1717449,1717626,1717691,1717735,1717780,1718744,1719047,1723064
 /qpid/trunk/qpid:796646-796653

Modified: 
qpid/java/branches/6.0.x/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/AMQPConnection_1_0.java
URL: 
http://svn.apache.org/viewvc/qpid/java/branches/6.0.x/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/AMQPConnection_1_0.java?rev=1723717&r1=1723716&r2=1723717&view=diff
==============================================================================
--- 
qpid/java/branches/6.0.x/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/AMQPConnection_1_0.java
 (original)
+++ 
qpid/java/branches/6.0.x/broker-plugins/amqp-1-0-protocol/src/main/java/org/apache/qpid/server/protocol/v1_0/AMQPConnection_1_0.java
 Fri Jan  8 12:34:06 2016
@@ -337,105 +337,115 @@ public class AMQPConnection_1_0 extends
                 Binary bin = new Binary(data);
                 RAW_LOGGER.debug("RECV[" + getNetwork().getRemoteAddress() + 
"] : " + bin.toString());
             }
-            switch(_state)
+            ProtocolHandler frameHandler;
+            int remaining;
+
+            do
             {
-                case A:
-                    if (msg.hasRemaining())
-                    {
-                        msg.get();
-                    }
-                    else
-                    {
-                        break;
-                    }
-                case M:
-                    if (msg.hasRemaining())
-                    {
-                        msg.get();
-                    }
-                    else
-                    {
-                        _state = State.M;
-                        break;
-                    }
+                frameHandler = _frameHandler;
+                remaining = msg.remaining();
 
-                case Q:
-                    if (msg.hasRemaining())
-                    {
-                        msg.get();
-                    }
-                    else
-                    {
-                        _state = State.Q;
-                        break;
-                    }
-                case P:
-                    if (msg.hasRemaining())
-                    {
-                        msg.get();
-                    }
-                    else
-                    {
-                        _state = State.P;
-                        break;
-                    }
-                case PROTOCOL:
-                    if (msg.hasRemaining())
-                    {
-                        msg.get();
-                    }
-                    else
-                    {
-                        _state = State.PROTOCOL;
-                        break;
-                    }
-                case MAJOR:
-                    if (msg.hasRemaining())
-                    {
-                        _major = msg.get();
-                    }
-                    else
-                    {
-                        _state = State.MAJOR;
-                        break;
-                    }
-                case MINOR:
-                    if (msg.hasRemaining())
-                    {
-                        _minor = msg.get();
-                    }
-                    else
-                    {
-                        _state = State.MINOR;
-                        break;
-                    }
-                case REVISION:
-                    if (msg.hasRemaining())
-                    {
-                        _revision = msg.get();
+                switch (_state)
+                {
+                    case A:
+                        if (msg.hasRemaining())
+                        {
+                            msg.get();
+                        }
+                        else
+                        {
+                            break;
+                        }
+                    case M:
+                        if (msg.hasRemaining())
+                        {
+                            msg.get();
+                        }
+                        else
+                        {
+                            _state = State.M;
+                            break;
+                        }
+
+                    case Q:
+                        if (msg.hasRemaining())
+                        {
+                            msg.get();
+                        }
+                        else
+                        {
+                            _state = State.Q;
+                            break;
+                        }
+                    case P:
+                        if (msg.hasRemaining())
+                        {
+                            msg.get();
+                        }
+                        else
+                        {
+                            _state = State.P;
+                            break;
+                        }
+                    case PROTOCOL:
+                        if (msg.hasRemaining())
+                        {
+                            msg.get();
+                        }
+                        else
+                        {
+                            _state = State.PROTOCOL;
+                            break;
+                        }
+                    case MAJOR:
+                        if (msg.hasRemaining())
+                        {
+                            _major = msg.get();
+                        }
+                        else
+                        {
+                            _state = State.MAJOR;
+                            break;
+                        }
+                    case MINOR:
+                        if (msg.hasRemaining())
+                        {
+                            _minor = msg.get();
+                        }
+                        else
+                        {
+                            _state = State.MINOR;
+                            break;
+                        }
+                    case REVISION:
+                        if (msg.hasRemaining())
+                        {
+                            _revision = msg.get();
 
-                        _state = State.FRAME;
-                    }
-                    else
-                    {
-                        _state = State.REVISION;
-                        break;
-                    }
-                case FRAME:
-                    if (msg.hasRemaining())
-                    {
-                        AccessController.doPrivileged(new 
PrivilegedAction<Void>()
+                            _state = State.FRAME;
+                        }
+                        else
+                        {
+                            _state = State.REVISION;
+                            break;
+                        }
+                    case FRAME:
+                        if (msg.hasRemaining())
                         {
-                            @Override
-                            public Void run()
+                            AccessController.doPrivileged(new 
PrivilegedAction<Void>()
                             {
-                                _frameHandler = _frameHandler.parse(msg);
-                                return null;
-                            }
-                        }, getAccessControllerContext());
+                                @Override
+                                public Void run()
+                                {
+                                    _frameHandler = _frameHandler.parse(msg);
+                                    return null;
+                                }
+                            }, getAccessControllerContext());
 
-                    }
+                        }
+                }
             }
+            while(_frameHandler != frameHandler || msg.remaining() != 
remaining);
         }
         catch(ConnectionScopedRuntimeException e)
         {



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to