Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes:
> @@ -967,7 +967,7 @@ CreateWellKnownSid( WELL_KNOWN_SID_TYPE WellKnownSidType,
> unsigned int i;
> TRACE("(%d, %s, %p, %p)\n", WellKnownSidType, debugstr_sid(DomainSid), pSid, cbSid);
>
> - if (cbSid == NULL || (DomainSid && !IsValidSid(DomainSid)))
> + if (cbSid == NULL || (DomainSid && !IsValidSid(DomainSid)) || WellKnownSidType == WinLogonIdsSid)
It doesn't look right to special-case it like this. It probably needs
more test cases to find out what's going on with such sids.
--
Alexandre Julliard
julliard(a)winehq.org