diff options
| author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-03-27 15:06:27 +0000 |
|---|---|---|
| committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-03-27 15:06:27 +0000 |
| commit | 3caee2835d5c42293619d92539e6b41473832c54 (patch) | |
| tree | 0dec6ceb8d1b512dc70d322281d63b2a61f89ad8 /lasso/id-ff/session.c | |
| parent | 70299b52ab264a2e1b7d1f5e8ea14135dbfbe183 (diff) | |
| download | lasso-3caee2835d5c42293619d92539e6b41473832c54.tar.gz lasso-3caee2835d5c42293619d92539e6b41473832c54.tar.xz lasso-3caee2835d5c42293619d92539e6b41473832c54.zip | |
ID-FF 1.2: fix some real and potential memory leaks
* lasso/id-ff/provider.c:
* lasso/id-ff/server.c:
* lasso/id-ff/session.c:
use macros to release previous value when necessary,
release object used as parameters to constructors,
free the encryption key associated with a provider,
release the key manager created for a saml signature
verification.
Diffstat (limited to 'lasso/id-ff/session.c')
| -rw-r--r-- | lasso/id-ff/session.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lasso/id-ff/session.c b/lasso/id-ff/session.c index 15774d7e..a4bd629b 100644 --- a/lasso/id-ff/session.c +++ b/lasso/id-ff/session.c @@ -456,8 +456,7 @@ lasso_session_get_assertion_identity_token(LassoSession *session, const gchar *s if (security_context->Token != NULL) { sec_token = security_context->Token->data; if (LASSO_IS_SAML2_ASSERTION(sec_token->any)) { - assertion = LASSO_SAML2_ASSERTION( - g_object_ref(sec_token->any)); + lasso_assign_gobject(assertion, sec_token->any); break; } } @@ -652,6 +651,7 @@ init_from_xml(LassoNode *node, xmlNode *xmlnode) assertion = lasso_node_new_from_xmlNode(n); lasso_session_add_assertion_simple(session, (char*)value, assertion); + lasso_release_gobject(assertion); xmlFree(value); } } else if ((value = xmlGetProp(t, (xmlChar*)"ID"))) { |