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

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about server/mapping.c:
>  return page_mask + 1;
>
> }
>
> +struct object *create_object_mapping( struct object *object, mem_size_t size, void **ptr )
> +{
> - static const unsigned int access = FILE_READ_DATA | FILE_WRITE_DATA;
> - struct mapping *mapping;
> - void *tmp;
> -
> - if (!(mapping = create_mapping( object, NULL, 0, size, SEC_COMMIT, 0, access, NULL ))) return NULL;
> - if ((tmp = mmap( NULL, mapping->size, PROT_READ | PROT_WRITE, MAP_SHARED, get_unix_fd( mapping->fd ), 0 )) == MAP_FAILED)

If the host and guest page sizes don't match, I think we should prefer the original size for mmap(2). Note that we also use original size for munmap(2).

```suggestion:-0+0
    if ((tmp = mmap( NULL, size, PROT_READ | PROT_WRITE, MAP_SHARED, get_unix_fd( mapping->fd ), 0 )) == MAP_FAILED)
```