You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2014/11/06 22:42:48 UTC
svn commit: r1637229 -
/pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
Author: lehmi
Date: Thu Nov 6 21:42:48 2014
New Revision: 1637229
URL: http://svn.apache.org/r1637229
Log:
PDFBOX-2469: avoid NPE as proposed by Tilman Hausherr
Modified:
pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
Modified: pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java?rev=1637229&r1=1637228&r2=1637229&view=diff
==============================================================================
--- pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java (original)
+++ pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java Thu Nov 6 21:42:48 2014
@@ -1444,8 +1444,9 @@ public class NonSequentialPDFParser exte
else if (pb instanceof COSDictionary)
{
COSDictionary dict = (COSDictionary) pb;
- // skip signature dictionary
- if (!dict.getItem(COSName.TYPE).equals(COSName.SIG))
+ COSBase type = dict.getItem(COSName.TYPE);
+ // skip dictionary containing the signature
+ if (type == null || !type.equals(COSName.SIG))
{
for (Entry<COSName, COSBase> entry : dict.entrySet())
{