excessive cpu load(Infinite)

[tsabbay]

We noticed 100% cpu load (infinite) when the data is containing 
 (
) Or similar codes for new line.
Example of code

const { DOMParser } = require('xmldom')

let xmlDocument = new DOMParser().parseFromString(documentFile)

Attached is a document, beware of parsing this as it's going to choke your instance.
hell end line.docx

Happy to provide more info.

[karfau]

Thank you for providing the test data. There are earlier similar reports but nobody was able to provide example data yet.

Do i understand correctly that the docx file should be read into a string and its content parsed? (I thought it's a zip file.)

[tsabbay]

Yes, took us time to identify this Read the file to string and try parse it.

…

On Wed, Sep 15, 2021, 22:50 Christian Bewernitz ***@***.***> wrote: Thank you for providing the test data. There are earlier similar reports but nobody was able to provide example data yet. Do i understand correctly that the docx file should be read into a string and its content parsed? (I thought it's a zip file.) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#324 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACPTZXBWHHRWVZT5K54T7YTUCD2JJANCNFSM5ECGPTDQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[karfau]

Hey there. I looked into it.
First of all as I thought, the docx file is basically a zip file, so xmldom can not "parse" it directly as your initial post suggested.
It contains the following structure:

├── [Content_Types].xml
├── docProps
│   ├── app.xml
│   └── core.xml
├── _rels
└── word
    ├── document.xml
    ├── fontTable.xml
    ├── _rels
    │   └── document.xml.rels
    ├── settings.xml
    ├── styles.xml
    ├── theme
    │   └── theme1.xml
    └── webSettings.xml

5 directories, 10 files

I searched the files for the character you mentioned, and only found it in word/document.xml.

Good news is: this is already fixed on master since we landed #314
We are currently working on the last two issues that will be part of the 0.8.0 release that will contain this bugfix. You can check on our progress in the related milestone