[PATCH 2/5] aclui: Populate the users list.
Zebediah Figura (she/her)
zfigura at codeweavers.com
Sun Feb 28 10:26:05 CST 2021
On 2/28/21 2:31 AM, Dmitry Timoshkov wrote:
> Zebediah Figura <z.figura12 at gmail.com> wrote:
>
>> +static PSID get_sid_from_ace(ACE_HEADER *ace)
>> +{
>> + switch (ace->AceType)
>> + {
>> + case ACCESS_ALLOWED_ACE_TYPE:
>> + return (SID *)&((ACCESS_ALLOWED_ACE *)ace)->SidStart;
>> + case ACCESS_DENIED_ACE_TYPE:
>> + return (SID *)&((ACCESS_DENIED_ACE *)ace)->SidStart;
>> + default:
>> + FIXME("Unhandled ACE type %#x.\n", ace->AceType);
>> + return NULL;
>> + }
>> +}
>
> I guess the cast (SID *) should be replaced by (PSID) since they are
> different types, and the compiler doesn't complain here because PSID
> is typedef'ed to PVOID.
>
Yeah, I think you're right; the casts are redundant and should probably
be removed.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20210228/62801066/attachment-0001.sig>
More information about the wine-devel
mailing list