summaryrefslogtreecommitdiff
path: root/samples
diff options
context:
space:
mode:
authorAxel Rasmussen <axelrasmussen@google.com>2021-05-15 03:27:19 +0300
committerLinus Torvalds <torvalds@linux-foundation.org>2021-05-15 05:41:32 +0300
commit7ed9d238c7dbb1fdb63ad96a6184985151b0171c (patch)
tree4a9dbcca365856ff5268c5174f5fa204f842ce22 /samples
parentd6e621de1fceb3b098ebf435ef7ea91ec4838a1a (diff)
downloadlinux-7ed9d238c7dbb1fdb63ad96a6184985151b0171c.tar.xz
userfaultfd: release page in error path to avoid BUG_ON
Consider the following sequence of events: 1. Userspace issues a UFFD ioctl, which ends up calling into shmem_mfill_atomic_pte(). We successfully account the blocks, we shmem_alloc_page(), but then the copy_from_user() fails. We return -ENOENT. We don't release the page we allocated. 2. Our caller detects this error code, tries the copy_from_user() after dropping the mmap_lock, and retries, calling back into shmem_mfill_atomic_pte(). 3. Meanwhile, let's say another process filled up the tmpfs being used. 4. So shmem_mfill_atomic_pte() fails to account blocks this time, and immediately returns - without releasing the page. This triggers a BUG_ON in our caller, which asserts that the page should always be consumed, unless -ENOENT is returned. To fix this, detect if we have such a "dangling" page when accounting fails, and if so, release it before returning. Link: https://lkml.kernel.org/r/20210428230858.348400-1-axelrasmussen@google.com Fixes: cb658a453b93 ("userfaultfd: shmem: avoid leaking blocks and used blocks in UFFDIO_COPY") Signed-off-by: Axel Rasmussen <axelrasmussen@google.com> Reported-by: Hugh Dickins <hughd@google.com> Acked-by: Hugh Dickins <hughd@google.com> Reviewed-by: Peter Xu <peterx@redhat.com> Cc: <stable@vger.kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'samples')
0 files changed, 0 insertions, 0 deletions