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;
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3103#note_63378