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 2016/10/20 18:05:28 UTC
svn commit: r1765853 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDNonTerminalField.java
Author: tilman
Date: Thu Oct 20 18:05:28 2016
New Revision: 1765853
URL: http://svn.apache.org/viewvc?rev=1765853&view=rev
Log:
PDFBOX-3534: avoid NPE if AcroForm field's child cosdict is null, as suggested by Tim Allison
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDNonTerminalField.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDNonTerminalField.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDNonTerminalField.java?rev=1765853&r1=1765852&r2=1765853&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDNonTerminalField.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDNonTerminalField.java Thu Oct 20 18:05:28 2016
@@ -135,10 +135,14 @@ public class PDNonTerminalField extends
COSArray kids = (COSArray)getCOSObject().getDictionaryObject(COSName.KIDS);
for (int i = 0; i < kids.size(); i++)
{
- PDField field = PDField.fromDictionary(getAcroForm(), (COSDictionary)kids.getObject(i), this);
- if (field != null)
+ COSDictionary cosDict = (COSDictionary)kids.getObject(i);
+ if (cosDict != null)
{
- children.add(field);
+ PDField field = PDField.fromDictionary(getAcroForm(), cosDict, this);
+ if (field != null)
+ {
+ children.add(field);
+ }
}
}
return children;