> From: Alexandre Julliard [mailto:[EMAIL PROTECTED] 
> 
> "Ge van Geldorp" <[EMAIL PROTECTED]> 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.



Reply via email to