Re: [PATCH v24 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:
> -
>        '_','_','w','i','n','e','_','s','e','s','s','i','o','n',0
> -
>    };
> -
>    UNICODE_STRING name = RTL_CONSTANT_STRING( nameW );
> -
>    OBJECT_ATTRIBUTES attr;
> -
>    unsigned int status;
> -
>    HANDLE handle;
> -
> -
>    if (!(session = calloc( 1, sizeof(*session) )))
> -
>    {
> -
>        pthread_mutex_unlock( &session_lock );
> -
>        return NULL;
> -
>    }
> -
>    session->ref = 1;
> -
> -
>    InitializeObjectAttributes( &attr, &name, 0, NULL, NULL );
> -
>    if (!(status = NtOpenSection( &handle, SECTION_MAP_READ | SECTION_QUERY, &attr )))

(nit) I don't think `SECTION_QUERY` is required for NtMapViewOfSection() to automatically infer the full size.

```suggestion:-0+0
        if (!(status = NtOpenSection( &handle, SECTION_MAP_READ, &attr )))
```