You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ra...@apache.org on 2010/01/08 22:19:26 UTC

svn commit: r897323 - /qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/util/Functions.java

Author: rajith
Date: Fri Jan  8 21:19:25 2010
New Revision: 897323

URL: http://svn.apache.org/viewvc?rev=897323&view=rev
Log:
There is mistake in a commit made for QPID-1830 in rev 891805
Due to the mistake a get operation could reference an index out of bounds or it could print fewer items than what is required.
This commit corrects that mistake.

Modified:
    qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/util/Functions.java

Modified: qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/util/Functions.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/util/Functions.java?rev=897323&r1=897322&r2=897323&view=diff
==============================================================================
--- qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/util/Functions.java (original)
+++ qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/util/Functions.java Fri Jan  8 21:19:25 2010
@@ -65,9 +65,9 @@
         StringBuilder str = new StringBuilder();
         str.append('"');
 
-        for (int i = 0; i < min(buf.remaining(), limit); i++)
+        for (int i = start; i < min(buf.limit(), limit); i++)
         {
-            byte c = buf.get(start + i);
+            byte c = buf.get(i);
 
             if (c > 31 && c < 127 && c != '\\')
             {



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org