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/08/29 09:48:23 UTC

svn commit: r570672 - in /webservices/rampart/trunk/c/src: omxmlsec/axiom.c omxmlsec/key.c omxmlsec/openssl/cipher_property.c util/rampart_sec_header_processor.c

Author: kaushalye
Date: Wed Aug 29 00:48:22 2007
New Revision: 570672

URL: http://svn.apache.org/viewvc?rev=570672&view=rev
Log:
Memory leak fix 
Build break fix in oxs_key_dup function

Modified:
    webservices/rampart/trunk/c/src/omxmlsec/axiom.c
    webservices/rampart/trunk/c/src/omxmlsec/key.c
    webservices/rampart/trunk/c/src/omxmlsec/openssl/cipher_property.c
    webservices/rampart/trunk/c/src/util/rampart_sec_header_processor.c

Modified: webservices/rampart/trunk/c/src/omxmlsec/axiom.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/omxmlsec/axiom.c?rev=570672&r1=570671&r2=570672&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/omxmlsec/axiom.c (original)
+++ webservices/rampart/trunk/c/src/omxmlsec/axiom.c Wed Aug 29 00:48:22 2007
@@ -239,8 +239,11 @@
             }
         }
     }
-
-    return NULL;
+    axutil_hash_free(attr_list, env);
+    attr_list = NULL;
+  
+    /*Not found! return NULL*/
+   return NULL;
 }
 
 

Modified: webservices/rampart/trunk/c/src/omxmlsec/key.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/omxmlsec/key.c?rev=570672&r1=570671&r2=570672&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/omxmlsec/key.c (original)
+++ webservices/rampart/trunk/c/src/omxmlsec/key.c Wed Aug 29 00:48:22 2007
@@ -117,7 +117,7 @@
 
 }
 
-AXIS2_EXTERN oxs_key_t *AXIS2_CALL
+oxs_key_t *AXIS2_CALL
 oxs_key_dup(oxs_key_t *key, 
         const axutil_env_t *env)
 {

Modified: webservices/rampart/trunk/c/src/omxmlsec/openssl/cipher_property.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/omxmlsec/openssl/cipher_property.c?rev=570672&r1=570671&r2=570672&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/omxmlsec/openssl/cipher_property.c (original)
+++ webservices/rampart/trunk/c/src/omxmlsec/openssl/cipher_property.c Wed Aug 29 00:48:22 2007
@@ -217,7 +217,9 @@
 
 
 axis2_status_t AXIS2_CALL
-openssl_cipher_property_free(openssl_cipher_property_t * cprop, const axutil_env_t *env)
+openssl_cipher_property_free(
+    openssl_cipher_property_t * cprop, 
+    const axutil_env_t *env)
 {
 
     AXIS2_ENV_CHECK(env, AXIS2_FAILURE);

Modified: webservices/rampart/trunk/c/src/util/rampart_sec_header_processor.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/util/rampart_sec_header_processor.c?rev=570672&r1=570671&r2=570672&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/util/rampart_sec_header_processor.c (original)
+++ webservices/rampart/trunk/c/src/util/rampart_sec_header_processor.c Wed Aug 29 00:48:22 2007
@@ -444,8 +444,9 @@
     /*Free*/
     oxs_asym_ctx_free(asym_ctx, env);
     asym_ctx = NULL;
-    /*oxs_key_free(decrypted_sym_key, env);
-    decrypted_sym_key = NULL;*/
+
+    /*Free the ref list*/
+    
 
     return AXIS2_SUCCESS;
 }