You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@trafficserver.apache.org by mt...@apache.org on 2010/05/19 15:36:08 UTC

svn commit: r946183 - /trafficserver/traffic/trunk/proxy/ControlMatcher.cc

Author: mturk
Date: Wed May 19 13:36:08 2010
New Revision: 946183

URL: http://svn.apache.org/viewvc?rev=946183&view=rev
Log:
Comment a possible memory leak

Modified:
    trafficserver/traffic/trunk/proxy/ControlMatcher.cc

Modified: trafficserver/traffic/trunk/proxy/ControlMatcher.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/ControlMatcher.cc?rev=946183&r1=946182&r2=946183&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/ControlMatcher.cc (original)
+++ trafficserver/traffic/trunk/proxy/ControlMatcher.cc Wed May 19 13:36:08 2010
@@ -588,6 +588,10 @@ template<class Data, class Result>
     strncat(config_file_path, config_file, sizeof(config_file_path) - strlen(config_file_path) - 1);
     xfree(config_file);
   }
+  // XXX: memory leak if flags have DONT_BUILD_TABLE ?
+  // else {
+  //  xfree(config_file)
+  // }
 
   reMatch = NULL;
   hostMatch = NULL;