diff options
author | Adrian Kummerländer | 2014-04-23 21:26:58 +0200 |
---|---|---|
committer | Adrian Kummerländer | 2014-04-23 21:26:58 +0200 |
commit | 525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf (patch) | |
tree | 6044c2876a55ed4c794bd577909225c49015d402 /src/support/error/error_capacitor.cc | |
parent | 30ae1cc66daf8acbb556ec761f96690a2fa74637 (diff) | |
download | InputXSLT-525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf.tar InputXSLT-525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf.tar.gz InputXSLT-525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf.tar.bz2 InputXSLT-525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf.tar.lz InputXSLT-525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf.tar.xz InputXSLT-525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf.tar.zst InputXSLT-525aa8c4dc3e23f0369c7edb1fb507ecad5f43bf.zip |
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
Diffstat (limited to 'src/support/error/error_capacitor.cc')
0 files changed, 0 insertions, 0 deletions