24 Jan
2008
24 Jan
'08
11:14 a.m.
From: Alexandre Julliard [mailto:julliard(a)winehq.org]
"Ge van Geldorp" <ge(a)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.