summaryrefslogtreecommitdiff
path: root/fs/mpage.c
diff options
context:
space:
mode:
authorAl Viro <viro@ZenIV.linux.org.uk>2012-03-07 09:16:35 +0400
committerLinus Torvalds <torvalds@linux-foundation.org>2012-03-10 06:59:59 +0400
commit86b62a2cb4fc09037bbce2959d2992962396fd7f (patch)
treeea45bb1de1f49862415b5643dea840318d02ee67 /fs/mpage.c
parent86e06008338e5712603613a0f6770500f79e83bd (diff)
downloadlinux-86b62a2cb4fc09037bbce2959d2992962396fd7f.tar.xz
aio: fix io_setup/io_destroy race
Have ioctx_alloc() return an extra reference, so that caller would drop it on success and not bother with re-grabbing it on failure exit. The current code is obviously broken - io_destroy() from another thread that managed to guess the address io_setup() would've returned would free ioctx right under us; gets especially interesting if aio_context_t * we pass to io_setup() points to PROT_READ mapping, so put_user() fails and we end up doing io_destroy() on kioctx another thread has just got freed... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Acked-by: Benjamin LaHaise <bcrl@kvack.org> Reviewed-by: Jeff Moyer <jmoyer@redhat.com> Cc: stable@vger.kernel.org Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/mpage.c')
0 files changed, 0 insertions, 0 deletions