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*/