You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2011/06/07 10:25:40 UTC
svn commit: r1132903 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java
Author: lehmi
Date: Tue Jun 7 08:25:40 2011
New Revision: 1132903
URL: http://svn.apache.org/viewvc?rev=1132903&view=rev
Log:
PDFBOX-1029: avoid ArrayIndexOutOfBoundsException if the pdf defines more labels than pages as proposed by karthick
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java?rev=1132903&r1=1132902&r2=1132903&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDPageLabels.java Tue Jun 7 08:25:40 2011
@@ -30,6 +30,7 @@ import org.apache.pdfbox.cos.COSArray;
import org.apache.pdfbox.cos.COSBase;
import org.apache.pdfbox.cos.COSDictionary;
import org.apache.pdfbox.cos.COSInteger;
+import org.apache.pdfbox.cos.COSName;
import org.apache.pdfbox.pdmodel.PDDocument;
/**
@@ -169,7 +170,7 @@ public class PDPageLabels implements COS
arr.add(COSInteger.get(i.getKey()));
arr.add(i.getValue());
}
- dict.setItem("Nums", arr);
+ dict.setItem(COSName.NUMS, arr);
return dict;
}
@@ -215,7 +216,10 @@ public class PDPageLabels implements COS
{
public void newLabel(int pageIndex, String label)
{
- map[pageIndex] = label;
+ if(pageIndex < doc.getNumberOfPages())
+ {
+ map[pageIndex] = label;
+ }
}
});
return map;