summaryrefslogtreecommitdiff
path: root/arch/mn10300
diff options
context:
space:
mode:
authorJames Hogan <james.hogan@imgtec.com>2017-03-31 11:14:02 +0100
committerJames Hogan <james.hogan@imgtec.com>2017-04-05 15:25:02 +0100
commit563ddc1076109f2b3f88e6d355eab7b6fd4662cb (patch)
treea28dd6bc34d9b593aa275e0a90beccf574f7fe6d /arch/mn10300
parentfb8ea062a8f2e85256e13f55696c5c5f0dfdcc8b (diff)
metag/usercopy: Zero rest of buffer from copy_from_user
Currently we try to zero the destination for a failed read from userland in fixup code in the usercopy.c macros. The rest of the destination buffer is then zeroed from __copy_user_zeroing(), which is used for both copy_from_user() and __copy_from_user(). Unfortunately we fail to zero in the fixup code as D1Ar1 is set to 0 before the fixup code entry labels, and __copy_from_user() shouldn't even be zeroing the rest of the buffer. Move the zeroing out into copy_from_user() and rename __copy_user_zeroing() to raw_copy_from_user() since it no longer does any zeroing. This also conveniently matches the name needed for RAW_COPY_USER support in a later patch. Fixes: 373cd784d0fc ("metag: Memory handling") Reported-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: James Hogan <james.hogan@imgtec.com> Cc: linux-metag@vger.kernel.org Cc: stable@vger.kernel.org
Diffstat (limited to 'arch/mn10300')
0 files changed, 0 insertions, 0 deletions