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;
 }