shell32: Support getting the shell folder path for users other than the current user and the default user [resend]

Ge van Geldorp ge at thinstall.com
Thu Jan 24 05:14:42 CST 2008


> From: Alexandre Julliard [mailto:julliard at winehq.org] 
> 
> "Ge van Geldorp" <ge at thinstall.com> writes:
> 
> > +    if (! CopySid(SidSize, Sid, UserInfo->User.Sid))
> > +    {
> > +        HeapFree(GetProcessHeap(), 0, Sid);
> > +        Sid = NULL;
> > +    }
> 
> You should return a string directly, there's no reason to 
> make a copy of the SID.

There's no way for the caller to free the memory if I return
UserInfo->User.Sid directly.

Ge van Geldorp.




More information about the wine-devel mailing list