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

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/win32u/winstation.c:
> +}
> -
>
> +static NTSTATUS map_shared_session_section( struct shared_session *session, HANDLE handle )
> +{
> - NTSTATUS status;
> - SIZE_T size = 0;
> - BOOL valid;
> -
> - while (!(status = NtMapViewOfSection( handle, GetCurrentProcess(), (void **)&session->shared, 0, 0,
> -
>                                      NULL, &size, ViewUnmap, 0, PAGE_READONLY )))
> - {
> -
>    session->object_capacity = (size - offsetof(session_shm_t, objects[0])) / sizeof(session_obj_t);
> -
>    SHARED_READ_BEGIN( session->shared, session_shm_t )
> -
>    {
> -
>        if ((valid = session->object_capacity == shared->object_capacity))
> -
>            session->id = shared->obj.id;

Ditto

```suggestion:-1+0
            valid = session->object_capacity == shared->object_capacity;
            session->id = shared->obj.id;
```