Please go ahead Sun On Wed, Apr 6, 2011 at 1:24 PM, Hui Shi <kalin....@gmail.com> wrote: > Could a gatekeeper please review this patch? > https://bugs.open64.net/show_bug.cgi?id=753 > There are two main issue in ConstraintGraph::processInito and > ConstraintGraph::processInitv > 1. In ConstraintGraph::processInito, mem_pool is initialized and not deleted > at early exit point. Cause some mem_pool not free. > 2. Recursively call ConstraintGraph::processInitValues(ST) without guard if > this ST is already processed. Cause allocate more unfreed mem_pool > When tracing this issue, also find another small issue. The points to local > variables allocate memory in malloc_mem_pool, need free. > > Fix > 1. Call mem_pool_delete in early exit in ConstraintGraph::processInito > 2. Check if an ST is already processed when recursively got ST’s initialized > point_to. > 3. Clear local points_to variable. > > ------------------------------------------------------------------------------ > Xperia(TM) PLAY > It's a major breakthrough. An authentic gaming > smartphone on the nation's most reliable network. > And it wants your games. > http://p.sf.net/sfu/verizon-sfdev > _______________________________________________ > Open64-devel mailing list > Open64-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/open64-devel > >
------------------------------------------------------------------------------ Xperia(TM) PLAY It's a major breakthrough. An authentic gaming smartphone on the nation's most reliable network. And it wants your games. http://p.sf.net/sfu/verizon-sfdev _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel