You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2012/03/13 22:47:50 UTC

svn commit: r1300370 - /commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/CharSequenceReader.java

Author: sebb
Date: Tue Mar 13 21:47:50 2012
New Revision: 1300370

URL: http://svn.apache.org/viewvc?rev=1300370&view=rev
Log:
Add check for negative offset

Modified:
    commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/CharSequenceReader.java

Modified: commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/CharSequenceReader.java
URL: http://svn.apache.org/viewvc/commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/CharSequenceReader.java?rev=1300370&r1=1300369&r2=1300370&view=diff
==============================================================================
--- commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/CharSequenceReader.java (original)
+++ commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/CharSequenceReader.java Tue Mar 13 21:47:50 2012
@@ -104,7 +104,7 @@ public class CharSequenceReader extends 
         if (array == null) {
             throw new NullPointerException("Character array is missing");
         }
-        if (length < 0 || (offset + length) > array.length) {
+        if (length < 0 || offset < 0 || (offset + length) > array.length) {
             throw new IndexOutOfBoundsException("Array Size=" + array.length +
                     ", offset=" + offset + ", length=" + length);
         }