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 12:05:22 UTC

svn commit: r572601 - in /webservices/rampart/trunk/c/src: omxmlsec/openssl/rsa.c util/rampart_timestamp_token.c

Author: kaushalye
Date: Tue Sep  4 03:05:18 2007
New Revision: 572601

URL: http://svn.apache.org/viewvc?rev=572601&view=rev
Log:
Memory leak fix in timestamp token

Modified:
    webservices/rampart/trunk/c/src/omxmlsec/openssl/rsa.c
    webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c

Modified: webservices/rampart/trunk/c/src/omxmlsec/openssl/rsa.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/omxmlsec/openssl/rsa.c?rev=572601&r1=572600&r2=572601&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/omxmlsec/openssl/rsa.c (original)
+++ webservices/rampart/trunk/c/src/omxmlsec/openssl/rsa.c Tue Sep  4 03:05:18 2007
@@ -72,6 +72,9 @@
     }
     oxs_buffer_populate(out, env, encrypted, ret);
 
+    AXIS2_FREE(env->allocator, encrypted);
+    encrypted = NULL;
+
     return ret;
 }
 

Modified: webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c
URL: http://svn.apache.org/viewvc/webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c?rev=572601&r1=572600&r2=572601&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c (original)
+++ webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c Tue Sep  4 03:05:18 2007
@@ -64,9 +64,13 @@
         {
             created_val = rampart_generate_time(env, 0);   /*Current time*/
             axiom_element_set_text(created_ele, env, created_val, created_node);
+           
+            AXIS2_FREE(env->allocator, created_val);
+            created_val = NULL;
         }
+
         /*Then we build Expires element*/
-        /*If ttl<0 then we dont build the expires element.*/
+        /*TIP: If ttl<0 then we dont build the expires element.*/
         if(ttl<0){
             AXIS2_LOG_INFO(env->log, " [rampart][ts] ttl is %d which is less then zero. Thus we do not build the expires element. ");
             return AXIS2_SUCCESS;
@@ -78,9 +82,13 @@
         {
             expires_val = rampart_generate_time(env, ttl);
             axiom_element_set_text(expires_ele, env, expires_val, expires_node);
+            
+            AXIS2_FREE(env->allocator, expires_val);
+            expires_val = NULL;
 
         }
     }
+    
 
     return AXIS2_SUCCESS;
 }/*rampart_build_timestamp_token*/