You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@activemq.apache.org by ch...@apache.org on 2013/01/23 15:54:16 UTC

svn commit: r1437427 - /activemq/trunk/activemq-amqp/src/main/java/org/apache/activemq/transport/amqp/AmqpProtocolConverter.java

Author: chirino
Date: Wed Jan 23 14:54:15 2013
New Revision: 1437427

URL: http://svn.apache.org/viewvc?rev=1437427&view=rev
Log:
Fixes AMQ-4258: Endless loop when received incorrect AMQP message

Applied patch sumbitted by Robert Balent.  Many thanks!


Modified:
    activemq/trunk/activemq-amqp/src/main/java/org/apache/activemq/transport/amqp/AmqpProtocolConverter.java

Modified: activemq/trunk/activemq-amqp/src/main/java/org/apache/activemq/transport/amqp/AmqpProtocolConverter.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-amqp/src/main/java/org/apache/activemq/transport/amqp/AmqpProtocolConverter.java?rev=1437427&r1=1437426&r2=1437427&view=diff
==============================================================================
--- activemq/trunk/activemq-amqp/src/main/java/org/apache/activemq/transport/amqp/AmqpProtocolConverter.java (original)
+++ activemq/trunk/activemq-amqp/src/main/java/org/apache/activemq/transport/amqp/AmqpProtocolConverter.java Wed Jan 23 14:54:15 2013
@@ -159,6 +159,7 @@ class AmqpProtocolConverter {
                 frame.moveHead(count);
             } catch (Throwable e) {
                 handleException(new AmqpProtocolException("Could not decode AMQP frame: " + frame, true, e));
+                return;
             }
             try {