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 2023/08/12 15:55:15 UTC

svn commit: r1911617 - /pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java

Author: tilman
Date: Sat Aug 12 15:55:15 2023
New Revision: 1911617

URL: http://svn.apache.org/viewvc?rev=1911617&view=rev
Log:
PDFBOX-5649: avoid NPE

Modified:
    pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java

Modified: pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java?rev=1911617&r1=1911616&r2=1911617&view=diff
==============================================================================
--- pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java (original)
+++ pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java Sat Aug 12 15:55:15 2023
@@ -598,6 +598,11 @@ public class DomXmpParser
         // Instantiate abstract structured type with hint from first element
         Element first = elements.get(0);
         PropertyType ctype = checkPropertyDefinition(xmp, DomHelper.getQName(first));
+        if (ctype == null)
+        {
+            throw new XmpParsingException(ErrorType.NoType, "ctype is null, first: " + first + 
+                    ", DomHelper.getQName(first): " + DomHelper.getQName(first));
+        }
         Types tt = ctype.type();
         AbstractStructuredType ast = instanciateStructured(tm, tt, descriptor.getLocalPart(), first.getNamespaceURI());