You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ma...@apache.org on 2016/04/25 15:13:46 UTC

[3/3] activemq-artemis git commit: ARTEMIS-500 - possible index out of bounds after amqp sasl handshake

ARTEMIS-500 - possible index out of bounds after amqp sasl handshake

https://issues.apache.org/jira/browse/ARTEMIS-500


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

Branch: refs/heads/master
Commit: 76d937fab3ee4c3ecccb772403f6e1ae89eecc05
Parents: abe5fd6
Author: Andy Taylor <an...@gmail.com>
Authored: Fri Apr 22 11:23:48 2016 +0100
Committer: Martyn Taylor <mt...@redhat.com>
Committed: Mon Apr 25 14:13:30 2016 +0100

----------------------------------------------------------------------
 .../java/org/proton/plug/handler/impl/ProtonHandlerImpl.java     | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/76d937fa/artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/handler/impl/ProtonHandlerImpl.java
----------------------------------------------------------------------
diff --git a/artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/handler/impl/ProtonHandlerImpl.java b/artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/handler/impl/ProtonHandlerImpl.java
index 5fd2a23..ed9a746 100644
--- a/artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/handler/impl/ProtonHandlerImpl.java
+++ b/artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/handler/impl/ProtonHandlerImpl.java
@@ -171,6 +171,10 @@ public class ProtonHandlerImpl extends ProtonInitializable implements ProtonHand
                try {
                   if (buffer.getByte(4) == 0x03) {
                      dispatchSASL();
+                     /*
+                     * there is a chance that if SASL Handshake has been carried out that the capacity may change.
+                     * */
+                     capacity = transport.capacity();
                   }
                }
                catch (Throwable ignored) {