From 525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf Mon Sep 17 00:00:00 2001
From: Adrian Kummerländer
Date: Wed, 23 Apr 2014 21:26:58 +0200
Subject: Fixed xerces DOM construction related memory leaks * Wrapped xerces
 DOM and support class instances in now DomDocumentGuard scope-guard class *
 FunctionReadDirectory class contains interal std::stack instance to store
 DomDocumentGuard instances ** wrapped in std::shared_ptr as
 FunctionReadDirectory is internally cloned by xalan... ** this is needed as
 the DOM has to preserved longer than the external function execution scope *
 Sadly XMLCh xerces strings have to be manually released ** added appropriate
 xercesc::XMLString::release calls * xalan::XercesDOMWrapperParsedSource does
 not mirror a given xerces DOM but convert it on instantiation ** this is why
 there is a dedicated finalize member method in InputXSLT::DomDocumentGuard *
 In short: I do not like the amount of trickery needed to simply prevent
 memory leaks in this context ** there sadly doesn't seem to be a
 substantially easier way to return arbitrary DOM trees from a external
 function

---
 src/support/dom_document_guard.h | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)
 create mode 100644 src/support/dom_document_guard.h

(limited to 'src/support/dom_document_guard.h')

diff --git a/src/support/dom_document_guard.h b/src/support/dom_document_guard.h
new file mode 100644
index 0000000..decca01
--- /dev/null
+++ b/src/support/dom_document_guard.h
@@ -0,0 +1,36 @@
+#ifndef INPUTXSLT_SRC_SUPPORT_DOM_DOCUMENT_GUARD_H_
+#define INPUTXSLT_SRC_SUPPORT_DOM_DOCUMENT_GUARD_H_
+
+#include <xalanc/XercesParserLiaison/XercesParserLiaison.hpp>
+#include <xalanc/XercesParserLiaison/XercesDOMSupport.hpp>
+#include <xalanc/XalanTransformer/XercesDOMWrapperParsedSource.hpp>
+
+#include <xercesc/dom/DOMDocument.hpp>
+
+#include <string>
+#include <memory>
+
+#include "common.h"
+
+namespace InputXSLT {
+
+class DomDocumentGuard {
+	public:
+		DomDocumentGuard(const std::string&);
+		~DomDocumentGuard();
+
+		xercesc::DOMDocument* operator->();
+		xalan::XalanDocument* finalize();
+
+	private:
+		xalan::XercesParserLiaison parser_;
+		xalan::XercesDOMSupport dom_support_;
+		XMLCh* root_node_name_;
+		xercesc::DOMDocument* const document_;
+		std::unique_ptr<xalan::XercesDOMWrapperParsedSource> parsed_source_;
+
+};
+
+}
+
+#endif  // INPUTXSLT_SRC_SUPPORT_DOM_DOCUMENT_GUARD_H_
-- 
cgit v1.2.3