Re: [PATCH] fixed potential double free

[Alexandre Julliard]

Marcus Meissner <marcus(a)jet.franken.de> 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
julliard(a)winehq.org

[PATCH] fixed potential double free

[Marcus Meissner]

On Thu, Jan 24, 2008 at 11:33:46AM +0100, Alexandre Julliard wrote:
> Marcus Meissner <marcus(a)jet.franken.de> 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.

I agree and adjusted it.

Ciao, Marcus