Re: [PATCH] fixed potential double free

2008-01-24 Thread Marcus Meissner
On Thu, Jan 24, 2008 at 11:33:46AM +0100, Alexandre Julliard wrote: > Marcus Meissner <[EMAIL PROTECTED]> writes: > > > Coverity spotted that set_window_pos() could call > > free_region() on the same pointer twice. > > > > (CID 671 and 672) > > > > Check that they are different before freeing them

Re: [PATCH] fixed potential double free

2008-01-24 Thread Alexandre Julliard
Marcus Meissner <[EMAIL PROTECTED]> writes: > Coverity spotted that set_window_pos() could call > free_region() on the same pointer twice. > > (CID 671 and 672) > > Check that they are different before freeing them twice. This can't happen, Coverity is confused. -- Alexandre Julliard [EMAIL PRO