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

Zebediah Figura zfigura at codeweavers.com
Tue Jun 28 11:54:54 CDT 2022


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.



More information about the wine-devel mailing list