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);
}