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( &section_str, mapping_nameW );
> - InitializeObjectAttributes( &attr, &section_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 )))
```