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 2022/09/14 03:48:08 UTC

svn commit: r1904054 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDDocumentCatalog.java

Author: tilman
Date: Wed Sep 14 03:48:08 2022
New Revision: 1904054

URL: http://svn.apache.org/viewvc?rev=1904054&view=rev
Log:
PDFBOX-5513: avoid IllegalArgumentException, as suggested by Karol Bryd 

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDDocumentCatalog.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDDocumentCatalog.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDDocumentCatalog.java?rev=1904054&r1=1904053&r2=1904054&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDDocumentCatalog.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDDocumentCatalog.java Wed Sep 14 03:48:08 2022
@@ -495,12 +495,16 @@ public class PDDocumentCatalog implement
         String mode = root.getNameAsString(COSName.PAGE_LAYOUT);
         if (mode != null)
         {
-            return PageLayout.fromString(mode);
-        }
-        else
-        {
-            return PageLayout.SINGLE_PAGE;
+            try
+            {
+                return PageLayout.fromString(mode);
+            }
+            catch (IllegalArgumentException e)
+            {
+                LOG.warn("Invalid PageLayout used '" + mode + "' - returning PageLayout.SINGLE_PAGE", e);
+            }
         }
+        return PageLayout.SINGLE_PAGE;
     }
 
     /**