Thread: [Sipp-commits] SF.net SVN: sipp:[510] sipp/trunk

SourceForge Headquarters 225 Broadway Suite 1600 San Diego, CA 92101 +1 (858) 454-5900

Revision: 510
          http://sipp.svn.sourceforge.net/sipp/?rev=510&view=rev
Author:   charlespwright
Date:     2008-07-18 11:09:18 +0000 (Fri, 18 Jul 2008)

Log Message:
-----------
Fix: Valgrind detected bug fixes.  Provided by Roland Meub.

Modified Paths:
--------------
    sipp/trunk/call.cpp
    sipp/trunk/scenario.cpp
    sipp/trunk/stat.cpp

Modified: sipp/trunk/call.cpp
===================================================================

--- sipp/trunk/call.cpp	2008-07-15 13:02:46 UTC (rev 509)
+++ sipp/trunk/call.cpp	2008-07-18 11:09:18 UTC (rev 510)
@@ -617,7 +617,9 @@
   if (use_tdmmap) {
     tdm_map[tdm_map_number] = false;
   }
-
+  
+  free(start_time_rtd);
+  free(rtd_done);
   free(debugBuffer);
 }
 

Modified: sipp/trunk/scenario.cpp
===================================================================
--- sipp/trunk/scenario.cpp	2008-07-15 13:02:46 UTC (rev 509)
+++ sipp/trunk/scenario.cpp	2008-07-18 11:09:18 UTC (rev 510)
@@ -685,9 +685,11 @@
     if(!strcmp(elem, "CallLengthRepartition")) {
       ptr = xp_get_string("value", "CallLengthRepartition");
       stats->setRepartitionCallLength(ptr);
+      free(ptr);
     } else if(!strcmp(elem, "ResponseTimeRepartition")) {
       ptr = xp_get_string("value", "ResponseTimeRepartition");
       stats->setRepartitionResponseTime(ptr);
+      free(ptr);
     } else if(!strcmp(elem, "Global")) {
       ptr = xp_get_string("variables", "Global");
 
@@ -699,6 +701,7 @@
 	globalVariables->find(currentTabVarName[i], true);
       }
       freeStringTable(currentTabVarName, currentNbVarNames);
+      free(ptr);
     } else if(!strcmp(elem, "User")) {
       ptr = xp_get_string("variables", "User");
 
@@ -710,6 +713,7 @@
 	userVariables->find(currentTabVarName[i], true);
       }
       freeStringTable(currentTabVarName, currentNbVarNames);
+      free(ptr);
     } else if(!strcmp(elem, "Reference")) {
       ptr = xp_get_string("variables", "Reference");
 
@@ -724,6 +728,7 @@
 	}
       }
       freeStringTable(currentTabVarName, currentNbVarNames);
+      free(ptr);
     } else if(!strcmp(elem, "DefaultMessage")) {
       char *id = xp_get_string("id", "DefaultMessage");
       if(!(ptr = xp_get_cdata())) {
@@ -739,6 +744,7 @@
 	ERROR("The label name '%s' is used twice.", ptr);
       }
       labelMap[ptr] = messages.size();
+      free(ptr);
     } else if (!strcmp(elem, "init")) {
       /* We have an init section, which must be full of nops or labels. */
       int nop_cursor = 0;
@@ -1370,6 +1376,7 @@
       tmpAction->setCaseIndep(xp_get_bool("case_indep", "ereg", false));
       tmpAction->setHeadersOnly(xp_get_bool("start_line", "ereg", false));
 
+      free(ptr);
       if ( 0 != ( ptr = xp_get_value((char *)"search_in") ) ) {
 	tmpAction->setOccurence(1);
 
@@ -1511,6 +1518,7 @@
       } else {
 	ERROR("Invalid 'compare' parameter: %s", ptr);
       }
+      free(ptr);
     } else if(!strcmp(actionElem, "verifyauth")) {
 #ifdef _USE_OPENSSL
       tmpAction->setVarId(xp_get_var("assign_to", "verifyauth"));

Modified: sipp/trunk/stat.cpp
===================================================================
--- sipp/trunk/stat.cpp	2008-07-15 13:02:46 UTC (rev 509)
+++ sipp/trunk/stat.cpp	2008-07-18 11:09:18 UTC (rev 510)
@@ -149,6 +149,7 @@
       delete [] M_ResponseTimeRepartition[i];
     }
   }
+  free(M_ResponseTimeRepartition);
 
   if (M_CallLengthRepartition != NULL)
     delete [] M_CallLengthRepartition;
@@ -174,6 +175,10 @@
    if(M_dumpRespTime != NULL)
      delete [] M_dumpRespTime ;
 
+  free(M_rtdInfo);
+  for (int_str_map::iterator i = M_revRtdMap.begin(); i != M_revRtdMap.end(); ++i) {
+		 free(i->second);
+  }
 
   M_SizeOfResponseTimeRepartition = 0;
   M_SizeOfCallLengthRepartition   = 0;
@@ -1064,7 +1069,8 @@
   M_counterDumpRespTime = 0          ; 
   M_dumpRespTime = NULL;
   M_fileNameRtt  = NULL;
-
+  M_rtdInfo = NULL;
+		 
   init();
 }
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




Thread: [Sipp-commits] SF.net SVN: sipp:[510] sipp/trunk

sipp-commits