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 2015/02/08 10:52:07 UTC

svn commit: r1658143 - /pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/BookmarkValidationProcess.java

Author: tilman
Date: Sun Feb  8 09:52:07 2015
New Revision: 1658143

URL: http://svn.apache.org/r1658143
Log:
PDFBOX-2630: add check that /Parent is an indirect object (should have been done in rev 1654609)

Modified:
    pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/BookmarkValidationProcess.java

Modified: pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/BookmarkValidationProcess.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/BookmarkValidationProcess.java?rev=1658143&r1=1658142&r2=1658143&view=diff
==============================================================================
--- pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/BookmarkValidationProcess.java (original)
+++ pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/process/BookmarkValidationProcess.java Sun Feb  8 09:52:07 2015
@@ -256,7 +256,11 @@ public class BookmarkValidationProcess e
     // verify that if certain named items exist, that they are indirect objects
     private boolean checkIndirectObjects(PreflightContext ctx, COSDictionary dictionary)
     {
-        // Prev, Next, First and Last must be indirect objects
+        // Parent, Prev, Next, First and Last must be indirect objects
+        if (!checkIndirectObject(ctx, dictionary, COSName.PARENT))
+        {
+            return false;
+        }
         if (!checkIndirectObject(ctx, dictionary, COSName.PREV))
         {
             return false;