"Vincent Povirk" <vincent at 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 at winehq.org