[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/33] linux-user: Propagate goto fail to return
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH 07/33] linux-user: Propagate goto fail to return |
Date: |
Mon, 4 Jun 2018 21:48:09 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
Le 01/06/2018 à 09:30, Richard Henderson a écrit :
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> linux-user/syscall.c | 62 ++++++++++++++++----------------------------
> 1 file changed, 23 insertions(+), 39 deletions(-)
>
> @@ -9951,18 +9947,15 @@ static abi_long do_syscall1(void *cpu_env, int num,
> abi_long arg1,
> case TARGET_SYSLOG_ACTION_READ_CLEAR: /* Read/clear msgs */
> case TARGET_SYSLOG_ACTION_READ_ALL: /* Read last messages */
> {
> - ret = -TARGET_EINVAL;
> if (len < 0) {
> - goto fail;
> + return -TARGET_EINVAL;
> }
> - ret = 0;
> if (len == 0) {
> - return ret;
> + return 0;
I think you should do this change in '[PATCH 02/33] linux-user: Relax
single exit from "break"'
Reviewed-by: Laurent Vivier <address@hidden>
- Re: [Qemu-devel] [PATCH 01/33] linux-user: Split out do_syscall1, (continued)
- [Qemu-devel] [PATCH 03/33] linux-user: Propagate goto ebadf to return, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 05/33] linux-user: Propagate goto unimplemented_nowarn to return, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 08/33] linux-user: Make syscall number unsigned, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 06/33] linux-user: Split out goto unimplemented to do_unimplemented, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 07/33] linux-user: Propagate goto fail to return, Richard Henderson, 2018/06/01
- Re: [Qemu-devel] [PATCH 07/33] linux-user: Propagate goto fail to return,
Laurent Vivier <=
- [Qemu-devel] [PATCH 04/33] linux-user: Propagate goto efault to return, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 12/33] linux-user: Split out open, openat, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 09/33] linux-user: Set up infrastructure for table-izing syscalls, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 10/33] linux-user: Split out brk, close, exit, read, write, Richard Henderson, 2018/06/01
- [Qemu-devel] [PATCH 02/33] linux-user: Relax single exit from "break", Richard Henderson, 2018/06/01