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 2021/05/28 15:49:18 UTC

svn commit: r1890289 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java

Author: tilman
Date: Fri May 28 15:49:18 2021
New Revision: 1890289

URL: http://svn.apache.org/viewvc?rev=1890289&view=rev
Log:
PDFBOX-4892: avoid NPE, as suggested by valerybokov

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java?rev=1890289&r1=1890288&r2=1890289&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fixup/processor/AcroFormOrphanWidgetsProcessor.java Fri May 28 15:49:18 2021
@@ -190,8 +190,10 @@ public class AcroFormOrphanWidgetsProces
         if (nonTerminalFieldsMap.get(parent.getString(COSName.T)) == null)
         {
             PDField field = PDFieldFactory.createField(acroForm, parent, null);
-            nonTerminalFieldsMap.put(field.getFullyQualifiedName(),field);
-
+            if (field != null)
+            {
+                nonTerminalFieldsMap.put(field.getFullyQualifiedName(), field);
+            }
             return field;
         }