]> git.karo-electronics.de Git - karo-tx-linux.git/commit
Prevent freeing uninitialized pointer in compat_do_readv_writev
authorDan Rosenberg <drosenberg@vsecurity.com>
Wed, 22 Sep 2010 18:32:56 +0000 (14:32 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 27 Sep 2010 00:18:28 +0000 (17:18 -0700)
commit32e987bf2db7941e5a27bfa4dbed50663c5b1ba2
tree6b95a75ed984ca6c13b532f2f077b69735760666
parente658aee24443e17c08424ae4ce193735c7fde737
Prevent freeing uninitialized pointer in compat_do_readv_writev

commit 767b68e96993e29e3480d7ecdd9c4b84667c5762 upstream.

In 32-bit compatibility mode, the error handling for
compat_do_readv_writev() may free an uninitialized pointer, potentially
leading to all sorts of ugly memory corruption.  This is reliably
triggerable by unprivileged users by invoking the readv()/writev()
syscalls with an invalid iovec pointer.  The below patch fixes this to
emulate the non-compat version.

Introduced by commit b83733639a49 ("compat: factor out
compat_rw_copy_check_uvector from compat_do_readv_writev")

Signed-off-by: Dan Rosenberg <dan.j.rosenberg@gmail.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/compat.c