DDXMLNode XMLStringWithOptions crash.

[ifgs1]

Hello, i am creating this issue since there is a crash that is continuously growing and we already identified.

The crash is due to a code that is using a function that is deprecated.

According to libxml2 library, we found that XMLNodeDump is deprecated based on: http://www.xmlsoft.org/html/libxml-tree.html#xmlNodeDump

"structures it is limited to 2GB and somehow deprecated, use xmlBufNodeDump() instead"

The issue is really hard to reproduce but we where able to track the root cause.

[chrisballinger]

This is a dupe of #105

What is your proposed patch and have you confirmed it fixes the crashes you are experiencing in production?

[ifgs1]

Hey @chrisballinger thanks for your quick response, so basically we are having these 2 crashes one in DDXMLNode.m - Line 1171 and the other on DDXMLNode.m line 1187. I am attaching screenshots for our crash logs:

[diegosalazarCO]

@chrisballinger According to libxml2 library you should be using xmlBufNodeDump() instead, but we have no certainty this will solve the issue, since right now we couldn't reproduce this issue. it seems to be related to #105.

We think it might be because of the way this functions try to convert the xml children, but tried with multiple XML stanzas and didn't succeed reproducing this.

[chrisballinger]

@ifgs1 @diegosalazarCO Have you tried running a patched version and seeing if it affects the crash rate? If it resolves the issue please submit a pull request.

Thank you!