diff options
| author | Frederic Peters <fpeters@entrouvert.com> | 2007-01-07 10:53:06 +0000 |
|---|---|---|
| committer | Frederic Peters <fpeters@entrouvert.com> | 2007-01-07 10:53:06 +0000 |
| commit | 8d4ac988a53409d5846dd2d3f4438c277de443de (patch) | |
| tree | dc960c149d85f17084b9856893d42a0a3323304d | |
| parent | 540e2ebabcf5678bf6980dfa8edfa81251f8424a (diff) | |
| download | lasso-8d4ac988a53409d5846dd2d3f4438c277de443de.tar.gz lasso-8d4ac988a53409d5846dd2d3f4438c277de443de.tar.xz lasso-8d4ac988a53409d5846dd2d3f4438c277de443de.zip | |
unfixed memory leak, because strange manipulations to xmlnode structures
could cause memory corruption
| -rw-r--r-- | lasso/xml/tools.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lasso/xml/tools.c b/lasso/xml/tools.c index 1795b8e3..2a3fa2b4 100644 --- a/lasso/xml/tools.c +++ b/lasso/xml/tools.c @@ -718,7 +718,12 @@ lasso_sign_node(xmlNode *xmlnode, const char *id_attr_name, const char *id_value xmlSecDSigCtxDestroy(dsig_ctx); xmlUnlinkNode(xmlnode); xmlnode->parent = old_parent; +#if 0 + /* memory leak since we don't free doc but it causes some little memory + * corruption; probably caused by the direct manipulation of xmlnode + * parent attribute. */ xmlFreeDoc(doc); +#endif return 0; } |