svn commit: r827991 - /webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c

[shankar]

Author: shankar
Date: Wed Oct 21 12:44:33 2009
New Revision: 827991

URL: http://svn.apache.org/viewvc?rev=827991&view=rev
Log:
Fix for issue RAMPARTC-139

Modified:
    webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c

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=827991&r1=827990&r2=827991&view=diff
==============================================================================
--- webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c (original)
+++ webservices/rampart/trunk/c/src/util/rampart_timestamp_token.c Wed Oct 21 
12:44:33 2009
@@ -172,12 +172,12 @@
     /* Check whether created is less than current time or not */
     current_val = rampart_generate_time(env, clock_skew_buffer, AXIS2_TRUE);
     validity = rampart_compare_date_time(env, current_val, created_val); 
-    AXIS2_FREE(env->allocator, current_val);
     if (validity == AXIS2_SUCCESS)
     {
         /* this means current_val < created_val. Which is not a valid case */
         AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, 
             "[rampart]Timestamp not valid: Created time is not valid");
+        AXIS2_FREE(env->allocator, current_val);
         return AXIS2_FAILURE;
     }
 
@@ -189,6 +189,7 @@
         /* If the expire element is not present, it means that the message 
will not be expired. */
         AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI,"[rampart]Cannot find expires 
in timestamp element."
             " This is not critical. Assume that the message is not expiring");
+        AXIS2_FREE(env->allocator, current_val);
         return AXIS2_SUCCESS;
     }
 
@@ -198,6 +199,7 @@
     {
         AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, "[rampart]Timestamp not valid: 
"
             "The second element of timestamp token (if any) MUST be EXPIRES");
+        AXIS2_FREE(env->allocator, current_val);
         return AXIS2_FAILURE;
     }
 
@@ -213,6 +215,17 @@
         /* this means created_value > expires_value. Which is not valid */
         AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, 
             "[rampart]Timestamp not valid: Timestamp token has expired");
+        AXIS2_FREE(env->allocator, current_val);
+        return AXIS2_FAILURE;
+    }
+
+    validity = rampart_compare_date_time(env, current_val, expires_val);
+    AXIS2_FREE(env->allocator, current_val);
+    if (validity == AXIS2_FAILURE)
+    {
+        /* this means current_value > expires_value. Which is not valid */
+        AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, 
+            "[rampart]Timestamp not valid: Timestamp token has expired");
         return AXIS2_FAILURE;
     }