svn commit: r1923378 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/documentnavigation/outline/PDOutlineNode.java

[tilman]

Author: tilman
Date: Sun Jan 26 15:50:34 2025
New Revision: 1923378

URL: http://svn.apache.org/viewvc?rev=1923378&view=rev
Log:
PDFBOX-5939: ignore outline that is its own parent

Modified:
    
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/documentnavigation/outline/PDOutlineNode.java

Modified: 
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/documentnavigation/outline/PDOutlineNode.java
URL: 
http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/documentnavigation/outline/PDOutlineNode.java?rev=1923378&r1=1923377&r2=1923378&view=diff
==============================================================================
--- 
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/documentnavigation/outline/PDOutlineNode.java
 (original)
+++ 
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/documentnavigation/outline/PDOutlineNode.java
 Sun Jan 26 15:50:34 2025
@@ -18,6 +18,9 @@ package org.apache.pdfbox.pdmodel.intera
 
 import java.util.Iterator;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 import org.apache.pdfbox.cos.COSBase;
 import org.apache.pdfbox.cos.COSDictionary;
 import org.apache.pdfbox.cos.COSName;
@@ -30,6 +33,7 @@ import org.apache.pdfbox.pdmodel.common.
  */
 public abstract class PDOutlineNode extends PDDictionaryWrapper
 {
+    private static final Log LOG = LogFactory.getLog(PDOutlineNode.class);
 
     /**
      * Default Constructor.
@@ -291,6 +295,12 @@ public abstract class PDOutlineNode exte
         PDOutlineNode parent = getParent();
         if (parent != null)
         {
+            if (getCOSObject() == parent.getCOSObject())
+            {
+                // PDFBOX-5939
+                LOG.warn("Outline parent points to itself");
+                return;
+            }
             if (parent.isNodeOpen())
             {
                 parent.setOpenCount(parent.getOpenCount() + delta);