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;


Reply via email to