diff options
author | Omar Sandoval <osandov@fb.com> | 2023-11-17 22:38:46 +0100 |
---|---|---|
committer | Christian Brauner <brauner@kernel.org> | 2023-11-18 16:42:07 +0100 |
commit | fe2c34bab6d46469ad3095955dc37e984dc24e38 (patch) | |
tree | 7e3ba304da7748545e8a5ec5cedab2baf58c6216 /fs/mnt_idmapping.c | |
parent | fs: Pass AT_GETATTR_NOSEC flag to getattr interface function (diff) | |
download | linux-fe2c34bab6d46469ad3095955dc37e984dc24e38.tar.xz linux-fe2c34bab6d46469ad3095955dc37e984dc24e38.zip |
iov_iter: fix copy_page_to_iter_nofault()
The recent conversion to inline functions made two mistakes:
1. It tries to copy the full amount requested (bytes), not just what's
available in the kmap'd page (n).
2. It's not applying the offset in the first page.
Note that copy_page_to_iter_nofault() is only used by /proc/kcore. This
was detected by drgn's test suite.
Fixes: f1982740f5e7 ("iov_iter: Convert iterate*() to inline funcs")
Signed-off-by: Omar Sandoval <osandov@fb.com>
Link: https://lore.kernel.org/r/c1616e06b5248013cbbb1881bb4fef85a7a69ccb.1700257019.git.osandov@fb.com
Acked-by: David Howells <dhowells@redhat.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'fs/mnt_idmapping.c')
0 files changed, 0 insertions, 0 deletions