PDFBOX-5951: ignore OpenAction that points to orphan page

THausherr · THausherr · commit af2f5a4d6342 · 2025-02-10T18:47:16.000Z
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1923708 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFMergerUtility.java b/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFMergerUtility.java
@@ -570,6 +570,11 @@ else if (openAction instanceof PDDestination)
                 if (page != null)
                 {
                     pageIndexOpenActionDest = srcCatalog.getPages().indexOf(page);
+                    if (pageIndexOpenActionDest == -1)
+                    {
+                        LOG.warn("OpenAction entry ignored because destination page doesn't exist");
+                        openAction = null;
+                    }
                 }
             }
 

Original file line number	Diff line number	Diff line change
`@@ -570,6 +570,11 @@ else if (openAction instanceof PDDestination)`
`570`	`570`	`if (page != null)`
`571`	`571`	`{`
`572`	`572`	`pageIndexOpenActionDest = srcCatalog.getPages().indexOf(page);`
	`573`	`+ if (pageIndexOpenActionDest == -1)`
	`574`	`+ {`
	`575`	`+ LOG.warn("OpenAction entry ignored because destination page doesn't exist");`
	`576`	`+ openAction = null;`
	`577`	`+ }`
`573`	`578`	`}`
`574`	`579`	`}`
`575`	`580`