You are viewing a plain text version of this content. The canonical link for it is here.
Posted to rampart-dev@ws.apache.org by ka...@apache.org on 2007/09/04 11:14:39 UTC
svn commit: r572589 - in /webservices/rampart/trunk/c/src/omxmlsec/openssl:
pkey.c sign.c
Author: kaushalye
Date: Tue Sep 4 02:14:38 2007
New Revision: 572589
URL: http://svn.apache.org/viewvc?rev=572589&view=rev
Log:
Memory leak fix in public key loading and signing process
Modified:
webservices/rampart/trunk/c/src/omxmlsec/openssl/pkey.c
webservices/rampart/trunk/c/src/omxmlsec/openssl/sign.c
Modified: webservices/rampart/trunk/c/src/omxmlsec/openssl/pkey.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/omxmlsec/openssl/pkey.c?rev=572589&r1=572588&r2=572589&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/omxmlsec/openssl/pkey.c (original)
+++ webservices/rampart/trunk/c/src/omxmlsec/openssl/pkey.c Tue Sep 4 02:14:38 2007
@@ -114,7 +114,8 @@
if (pkey->key)
{
- AXIS2_FREE(env->allocator, pkey->key);
+ /* AXIS2_FREE(env->allocator, pkey->key);*/
+ EVP_PKEY_free(pkey->key);
pkey->key = NULL;
}
@@ -232,7 +233,8 @@
if (pkey->key)
{
- AXIS2_FREE(env->allocator, pkey->key);
+ /*AXIS2_FREE(env->allocator, pkey->key);*/
+ EVP_PKEY_free(pkey->key);
pkey->key = NULL;
}
if (pkey->name)
Modified: webservices/rampart/trunk/c/src/omxmlsec/openssl/sign.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/omxmlsec/openssl/sign.c?rev=572589&r1=572588&r2=572589&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/omxmlsec/openssl/sign.c (original)
+++ webservices/rampart/trunk/c/src/omxmlsec/openssl/sign.c Tue Sep 4 02:14:38 2007
@@ -72,6 +72,8 @@
/*Fill the output buffer*/
oxs_buffer_populate(output_buf, env, sig_buf, sig_len);
+ EVP_MD_CTX_cleanup(&md_ctx);
+
return sig_len;
}