Re: [PATCH v3 0/1] MR3390: ntdll: Correctly handle pread short reads

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/ntdll/unix/virtual.c:
>          (vprot & VPROT_COMMITTED))
>      {
>          if (pagemap_fd == -1 ||
> -
>            pread( pagemap_fd, &pagemap, sizeof(pagemap), ((UINT_PTR)p->VirtualAddress >> page_shift) * sizeof(pagemap) ) != sizeof(pagemap))
> -
>            pread_all( pagemap_fd, &pagemap, sizeof(pagemap), ((UINT_PTR)p->VirtualAddress >> page_shift) * sizeof(pagemap) ) != sizeof(pagemap))

Partial reads on `/proc/<pid>/pagemap` should *not* be handled at all. Unaligned offset (not multiples of `sizeof(pagemap)`) results in `-EINVAL` anyway.

```suggestion:-0+0
                pread( pagemap_fd, &pagemap, sizeof(pagemap), ((UINT_PTR)p->VirtualAddress >> page_shift) * sizeof(pagemap) ) != sizeof(pagemap))
```