[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [6/6] shell32: implement Get/SetIDList on ShellItem objects
From:       Alexandre Julliard <julliard () winehq ! org>
Date:       2009-02-24 14:31:05
Message-ID: 87skm3x5xy.fsf () wine ! dyndns ! org
[Download RAW message or body]

"Vincent Povirk" <vincent@codeweavers.com> writes:

> +    ILFree(This->pidl);
> +    This->pidl = ILClone(pidl);
> +    if (This->pidl)
> +        return S_OK;
> +    else
> +        return E_OUTOFMEMORY;

You should check for success before you free the previous pidl.

-- 
Alexandre Julliard
julliard@winehq.org


[prev in list] [next in list] [prev in thread] [next in thread]