Author: ycwu
Date: 2011-07-08 03:47:03 -0400 (Fri, 08 Jul 2011)
New Revision: 3682

Modified:
   trunk/osprey/be/cg/hb_sched.cxx
Log:
priority_fn has a member __curbb_list witch is a
std::list, it would not use the memory pool _hb_pool. 
CXX_DELETE has to be called to free the memory of this member. 

code review by Lai Jianxin


Modified: trunk/osprey/be/cg/hb_sched.cxx
===================================================================
--- trunk/osprey/be/cg/hb_sched.cxx     2011-07-06 19:11:17 UTC (rev 3681)
+++ trunk/osprey/be/cg/hb_sched.cxx     2011-07-08 07:47:03 UTC (rev 3682)
@@ -2951,6 +2951,9 @@
 #if defined (TARG_SL)
   LOCS_Fwd_Scheduling = org_LOCS_Fwd_Scheduling;
 #endif
+  // cycle_fn will be deleted when _hb_pool is freed, while priority_fn used 
malloc pool
+  // to initialize its memeber _curbb_list, explicitly delete it to aviod 
memory leak.
+  CXX_DELETE(priority_fn, &_hb_pool);
 }
 
 void


------------------------------------------------------------------------------
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security 
threats, fraudulent activity, and more. Splunk takes this data and makes 
sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2d-c2
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to