NPE when trying to parse document with DTD declaration

[fasterxml]

NPE when trying to parse document with DTD declaration

[cowtowncoder]

Aalto should handle such documents fine, skipping DTDs.

So a reproduction would be needed to show NPE case.

[borkdude]

@cowtowncoder Sorry for responding so late, but I finally bumped into this error again and have a minimal reproducible XML now.

Repro with Clojure (where I'm using aalto from):

The XML (in /tmp/xml.xml):

<?xml version="1.0" encoding="US-ASCII"?>
<!DOCTYPE issue-xml PUBLIC "-//Atypon//DTD Atypon JATS Journal Archiving and Interchange Issue XML" "Atypon-Issue-Xml.dtd">
<issue-xml>
</issue-xml>

Without aalto-xml:

$ clojure -Sdeps '{:deps {org.clojure/data.xml {:mvn/version "RELEASE"}}}'
Clojure 1.10.0
user=> (require '[clojure.data.xml :as xml])
nil
user=> (xml/emit-str (xml/parse-str (slurp "/tmp/xml.xml")))
"<?xml version=\"1.0\" encoding=\"UTF-8\"?><issue-xml>\n</issue-xml>"
user=>

With aalto-xml:

$ clojure -Sdeps '{:deps {org.clojure/data.xml {:mvn/version "RELEASE"} com.fasterxml/aalto-xml {:mvn/version "RELEASE"}}}'
Clojure 1.10.0
user=> (require '[clojure.data.xml :as xml])
nil
user=> (xml/emit-str (xml/parse-str (slurp "/tmp/xml.xml")))
Execution error (NullPointerException) at com.fasterxml.aalto.util.TextBuilder/finishCurrentSegment (TextBuilder.java:707).
null

[cowtowncoder]

@borkdude Thank you.

[cowtowncoder]

Was an easy fix with reproduction, thanks!

I am now releasing 1.2.1, should be available soon.

[borkdude]

Thank you!