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

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/win32u/winstation.c:
> +static volatile void *map_object_shared_memory( HANDLE object, SIZE_T size )
> +{
> +    static const WCHAR mapping_nameW[] =
> +    {
> +        '_','_','w','i','n','e','_','m','a','p','p','i','n','g',0
> +    };
> +    OBJECT_ATTRIBUTES attr;
> +    UNICODE_STRING section_str;
> +    HANDLE handle;
> +    UINT status;
> +    void *ptr;
> +
> +    RtlInitUnicodeString( §ion_str, mapping_nameW );
> +    InitializeObjectAttributes( &attr, §ion_str, 0, object, NULL );
> +    if (!(status = NtOpenSection( &handle, SECTION_ALL_ACCESS, &attr )))

(nit) excessive permissions

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

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