svn commit: r1930736 - pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml

[tilman]

Author: tilman
Date: Fri Dec 19 15:25:28 2025
New Revision: 1930736

Log:
PDFBOX-5288: fix push/pop imbalance

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
==============================================================================
--- 
pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java
    Fri Dec 19 15:25:24 2025        (r1930735)
+++ 
pdfbox/branches/2.0/xmpbox/src/main/java/org/apache/xmpbox/xml/DomXmpParser.java
    Fri Dec 19 15:25:28 2025        (r1930736)
@@ -505,15 +505,22 @@ public class DomXmpParser
             Element inner = DomHelper.getFirstChildElement(property);
             if (inner != null)
             {
-                nsFinder.push(inner);
-                AbstractStructuredType ast = parseLiDescription(xmp, 
DomHelper.getQName(property), inner);
-                if (ast == null)
+                try
                 {
-                    throw new XmpParsingException(ErrorType.Format, "inner 
element should contain child elements : "
-                            + inner);
+                    nsFinder.push(inner);
+                    AbstractStructuredType ast = parseLiDescription(xmp, 
DomHelper.getQName(property), inner);
+                    if (ast == null)
+                    {
+                        throw new XmpParsingException(ErrorType.Format, "inner 
element should contain child elements : "
+                                + inner);
+                    }
+                    ast.setPrefix(prefix);
+                    container.addProperty(ast);
+                }
+                finally
+                {
+                    nsFinder.pop();
                 }
-                ast.setPrefix(prefix);
-                container.addProperty(ast);
             }
         }
     }