[PATCH v3 0/4] MR313: win32u: Finish moving raw input APIs.

Zebediah Figura zfigura at codeweavers.com
Wed Jun 29 18:24:24 CDT 2022


On 6/28/22 12:24, Rémi Bernon (@rbernon) wrote:
> On Tue Jun 28 16:55:07 2022 +0000, **** wrote:
>> Zebediah Figura replied on the mailing list:
>> ```
>> On 6/28/22 02:10, Rémi Bernon (@rbernon) wrote:
>>> Rémi Bernon (@rbernon) commented about dlls/wow64win/user.c:
>>>> +
>>>> +    if (size != sizeof(RAWINPUTDEVICELIST32))
>>>> +    {
>>>> +        SetLastError( ERROR_INVALID_PARAMETER );
>>>> +        return ~0u;
>>>> +    }
>>>> +
>>>> +    if (devices32)
>>>> +    {
>>>> +        RAWINPUTDEVICELIST *devices64;
>>>> +        unsigned int ret, i;
>>>> +
>>>> +        if (!(devices64 = Wow64AllocateTemp( (*count) *
>> sizeof(*devices64) )))
>>>> +        {
>>>> +            SetLastError( ERROR_NOT_ENOUGH_MEMORY );
>>>> +            return FALSE;
>>> This should be ~0u.
>>>
>> The bridge doesn't give me any context on which commit is being
>> responded to (is it not possible to reconstruct that information?) but
>> RegisterRawInputDevices() returns BOOL, so one of those three is correct.
>> The other two are errors; thanks for finding them. I've resent a new
>> version with them fixed.
>> ```
> It's in wow64_NtUserGetRawInputDeviceList, in PATCH 3.
> 

Oops, indeed. I forgot that there were more than three functions that I 
converted...

Sent a new version with that one fixed as well.



More information about the wine-devel mailing list