Re: [PATCH v21 0/9] MR3103: server: Create and use a desktop shared mapping for GetCursorPos.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/win32u/winstation.c:
> -
>        return reply->index;
> -
>    }
> -
>    SERVER_END_REQ;
> -
>    break;
> - }
> -
> - return -1;
>
> +}
> -
>
> +/* return a strong reference on the last known session object for a thread id and type */
> +static struct session_object *get_thread_session_object( UINT tid, enum object_type type )
> +{
> - struct shared_session *session;
> - struct session_object *object;
> - BOOL valid = TRUE;
> - int index;

[-Wpedantic nit] `comparison between signed and unsigned` at `valid = index < session->object_capacity`

```suggestion:-0+0
    UINT index;
```

This will also ensure we won't get negative indices on overflow (unlikely, hence nit).