]> git.proxmox.com Git - mirror_qemu.git/commit
linux-user/sparc: Remove unneeded checks of 'err' from sparc64_get_context()
authorPeter Maydell <peter.maydell@linaro.org>
Fri, 6 Nov 2020 15:27:36 +0000 (15:27 +0000)
committerLaurent Vivier <laurent@vivier.eu>
Fri, 18 Dec 2020 10:23:29 +0000 (11:23 +0100)
commit309abce23658c0142430440c7ad1be2a02170ed0
treed24d64654f60c7683afda9967d0ebc93b85ef944
parent246ff44295d0153d1d272b4d4a77c4bf9cc6ad66
linux-user/sparc: Remove unneeded checks of 'err' from sparc64_get_context()

Unlike the kernel macros, our __get_user() and __put_user() do not
return a failure code.  Kernel code typically has a style of
  err |= __get_user(...); err |= __get_user(...);
and then checking err at the end.  In sparc64_get_context() our
version of the code dropped the accumulating into err but left the
"if (err) goto do_sigsegv" checks, which will never be taken. Delete
unnecessary if()s.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20201106152738.26026-3-peter.maydell@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
linux-user/sparc/signal.c