You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2017/03/06 17:33:27 UTC

svn commit: r1785699 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/filter/CCITTFaxDecoderStream.java

Author: tilman
Date: Mon Mar  6 17:33:27 2017
New Revision: 1785699

URL: http://svn.apache.org/viewvc?rev=1785699&view=rev
Log:
PDFBOX-3702: avoid ArrayIndexOutOfBoundsException

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/filter/CCITTFaxDecoderStream.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/filter/CCITTFaxDecoderStream.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/filter/CCITTFaxDecoderStream.java?rev=1785699&r1=1785698&r2=1785699&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/filter/CCITTFaxDecoderStream.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/filter/CCITTFaxDecoderStream.java Mon Mar  6 17:33:27 2017
@@ -189,7 +189,10 @@ final class CCITTFaxDecoderStream extend
 
                             runLength = decodeRun(white ? blackRunTree : whiteRunTree);
                             index += runLength;
-                            changesCurrentRow[changesCurrentRowCount++] = index;
+                            if (changesCurrentRowCount < changesCurrentRow.length)
+                            {
+                                changesCurrentRow[changesCurrentRowCount++] = index;
+                            }
                             break;
 
                         case VALUE_PASSMODE: