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

Reply via email to