Re: [v2 PATCH 1/2] advapi32: Handle WinLogonIdsSid in CreateWellKnownSid

[Alexandre Julliard]

Alistair Leslie-Hughes  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.