From nobody Wed Nov 5 14:29:49 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1534900801548902.7060655407155; Tue, 21 Aug 2018 18:20:01 -0700 (PDT) Received: from localhost ([::1]:56436 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fsHoW-0001TW-8c for importer@patchew.org; Tue, 21 Aug 2018 21:20:00 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36502) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fsHk3-00073T-V7 for qemu-devel@nongnu.org; Tue, 21 Aug 2018 21:15:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fsHjv-0001uv-4g for qemu-devel@nongnu.org; Tue, 21 Aug 2018 21:15:20 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:56341) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fsHjs-0001sb-Tg for qemu-devel@nongnu.org; Tue, 21 Aug 2018 21:15:14 -0400 Received: from localhost.localdomain ([78.238.229.36]) by mrelayeu.kundenserver.de (mreue102 [212.227.15.183]) with ESMTPSA (Nemesis) id 0LqDBo-1gMvZx17tg-00drZB; Wed, 22 Aug 2018 03:14:33 +0200 From: Laurent Vivier To: qemu-devel@nongnu.org Date: Wed, 22 Aug 2018 03:14:17 +0200 Message-Id: <20180822011417.11608-8-laurent@vivier.eu> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20180822011417.11608-1-laurent@vivier.eu> References: <20180822011417.11608-1-laurent@vivier.eu> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:0qS5GIr9Fi8mI0LMkv+piT5sWGwD9bDmFzfvM15krOPIbCAvOKb nh8HEtxYH31lXXK34Uj/riaXWR/fEKAZUvUzvAUwh7xqjm/0thxrjLGrjcf1R8wRFn2Wsii bKGciMMX2cZdpkonVP1kYacyxdecClu3ezQCwoRJoWIpz+En/aPb/PPfKEepfBEBqfkLxN1 sopLz+lDEQHjk/rjEvV6A== X-UI-Out-Filterresults: notjunk:1;V01:K0:krBQuI7pvD8=:TvowydZ/Hu19oSwoZd1bnP TYtFybcTuRO2ZWeYQy2ebf8pYbGXg9aJ8rxXSs+UCmXoty4TWvAeU2bkuPDLyENhzWjY5Fh6V QDAfnvYIjzYluw8ZCO1cCbCfaDSZIGrXz53Q9EV/Ee222Ic1BYIHH3P/LuMm3jwKCty11Ntyk ZLgUaDdU2yxETSZSWov0PYpfmSglWGfcMFHCQbjuQpm/bicUz04QC4CQ6TUeTmwprJTVR0LoD ZSNXHQf1FpViiEx3H8QiF1EaA3ipgS1+L8gGunS9qf+5i/RXj+kxAmd/tlYsfPbk5W2BytTby MjoDlvdHqeJdqW6O7Dklp3ERtSYuMAmC2ZMJ2SFvNcGNF1QbplwOmql/cYlDDdNKLxbiHNR8n pwgxBYrR+I4y+8kWBBUEQum9DIi28WqlnYZbSWJ+fzTPwtm3Hcnaydic+FSF55tqxJ6zPiHJt nh3b0QgqUxR0Lmgh8In6Caxbt3sXcyymKa4G/AzWa0htNdQTCoP0g0D18aqdPfrU9nNBm/RIs 0lEIUfBQrs4m5eSEGosExJZ2Vs+QN8MdaJZrBJlMia/nSmMYzbq007aq2SWgujYGBMG9X6T4N PSa/vcGLsg1IQ+JBNqY0vBmxjHWoo5xUlTVcm0slBtVrPXtgANIrh7zMy7Kee22aOWFwihsw4 89s0zJXkUf25NduBMdSt5UAX0DYe82zXKxGuIWykK/J6U1njN4fDuJ+6VE+aXoG9n5tJ0dQx1 FpoR4NvH8NiBDorP X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 212.227.17.13 Subject: [Qemu-devel] [PULL 7/7] linux-user: Propagate goto fail to return X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Riku Voipio , Richard Henderson , Laurent Vivier Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 From: Richard Henderson Reviewed-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Laurent Vivier Signed-off-by: Richard Henderson Message-Id: <20180818190118.12911-8-richard.henderson@linaro.org> Signed-off-by: Laurent Vivier --- linux-user/syscall.c | 60 ++++++++++++++++---------------------------- 1 file changed, 22 insertions(+), 38 deletions(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 57044cf546..02fba7606d 100644 --- a/linux-user/syscall.c +++ b/linux-user/syscall.c @@ -9016,8 +9016,7 @@ static abi_long do_syscall1(void *cpu_env, int num, a= bi_long arg1, how =3D SIG_SETMASK; break; default: - ret =3D -TARGET_EINVAL; - goto fail; + return -TARGET_EINVAL; } mask =3D arg2; target_to_host_old_sigset(&set, &mask); @@ -9044,8 +9043,7 @@ static abi_long do_syscall1(void *cpu_env, int num, a= bi_long arg1, how =3D SIG_SETMASK; break; default: - ret =3D -TARGET_EINVAL; - goto fail; + return -TARGET_EINVAL; } if (!(p =3D lock_user(VERIFY_READ, arg2, sizeof(target_sig= set_t), 1))) return -TARGET_EFAULT; @@ -9088,8 +9086,7 @@ static abi_long do_syscall1(void *cpu_env, int num, a= bi_long arg1, how =3D SIG_SETMASK; break; default: - ret =3D -TARGET_EINVAL; - goto fail; + return -TARGET_EINVAL; } if (!(p =3D lock_user(VERIFY_READ, arg2, sizeof(target_sig= set_t), 1))) return -TARGET_EFAULT; @@ -9379,15 +9376,15 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, =20 ret =3D copy_from_user_fdset_ptr(&rfds, &rfds_ptr, rfd_addr, n= ); if (ret) { - goto fail; + return ret; } ret =3D copy_from_user_fdset_ptr(&wfds, &wfds_ptr, wfd_addr, n= ); if (ret) { - goto fail; + return ret; } ret =3D copy_from_user_fdset_ptr(&efds, &efds_ptr, efd_addr, n= ); if (ret) { - goto fail; + return ret; } =20 /* @@ -9420,8 +9417,7 @@ static abi_long do_syscall1(void *cpu_env, int num, a= bi_long arg1, sig.set =3D &set; if (arg_sigsize !=3D sizeof(*target_sigset)) { /* Like the kernel, we enforce correct size sigset= s */ - ret =3D -TARGET_EINVAL; - goto fail; + return -TARGET_EINVAL; } target_sigset =3D lock_user(VERIFY_READ, arg_sigset, sizeof(*target_sigset), 1); @@ -9860,17 +9856,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 =3D -TARGET_EINVAL; if (len < 0) { - goto fail; + return -TARGET_EINVAL; } if (len =3D=3D 0) { return 0; } p =3D lock_user(VERIFY_WRITE, arg2, arg3, 0); if (!p) { - ret =3D -TARGET_EFAULT; - goto fail; + return -TARGET_EFAULT; } ret =3D get_errno(sys_syslog((int)arg1, p, (int)arg3)); unlock_user(p, arg2, arg3); @@ -10215,8 +10209,7 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, =20 dirp =3D g_try_malloc(count); if (!dirp) { - ret =3D -TARGET_ENOMEM; - goto fail; + return -TARGET_ENOMEM; } =20 ret =3D get_errno(sys_getdents(arg1, dirp, count)); @@ -10616,7 +10609,7 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, arg2 ? &node : NULL, NULL)); if (is_error(ret)) { - goto fail; + return ret; } if (arg1 && put_user_u32(cpu, arg1)) { return -TARGET_EFAULT; @@ -11072,8 +11065,7 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, grouplist =3D alloca(gidsetsize * sizeof(gid_t)); target_grouplist =3D lock_user(VERIFY_READ, arg2, gidsetsi= ze * sizeof(target_id), 1); if (!target_grouplist) { - ret =3D -TARGET_EFAULT; - goto fail; + return -TARGET_EFAULT; } for (i =3D 0; i < gidsetsize; i++) { grouplist[i] =3D low2highgid(tswapid(target_grouplist[= i])); @@ -11331,8 +11323,7 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, how =3D SIG_SETMASK; break; default: - ret =3D -TARGET_EINVAL; - goto fail; + return -TARGET_EINVAL; } mask =3D arg2; target_to_host_old_sigset(&set, &mask); @@ -11380,8 +11371,7 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, if (!is_error(ret)) { target_grouplist =3D lock_user(VERIFY_WRITE, arg2, gidsets= ize * 4, 0); if (!target_grouplist) { - ret =3D -TARGET_EFAULT; - goto fail; + return -TARGET_EFAULT; } for(i =3D 0;i < ret; i++) target_grouplist[i] =3D tswap32(grouplist[i]); @@ -11401,8 +11391,7 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, grouplist =3D alloca(gidsetsize * sizeof(gid_t)); target_grouplist =3D lock_user(VERIFY_READ, arg2, gidsetsize *= 4, 1); if (!target_grouplist) { - ret =3D -TARGET_EFAULT; - goto fail; + return -TARGET_EFAULT; } for(i =3D 0;i < gidsetsize; i++) grouplist[i] =3D tswap32(target_grouplist[i]); @@ -11477,20 +11466,17 @@ static abi_long do_syscall1(void *cpu_env, int nu= m, abi_long arg1, #ifdef TARGET_NR_mincore case TARGET_NR_mincore: { - void *a; - ret =3D -TARGET_ENOMEM; - a =3D lock_user(VERIFY_READ, arg1, arg2, 0); + void *a =3D lock_user(VERIFY_READ, arg1, arg2, 0); if (!a) { - goto fail; + return -TARGET_ENOMEM; } - ret =3D -TARGET_EFAULT; p =3D lock_user_string(arg3); if (!p) { - goto mincore_fail; + ret =3D -TARGET_EFAULT; + } else { + ret =3D get_errno(mincore(a, arg2, p)); + unlock_user(p, arg3, ret); } - ret =3D get_errno(mincore(a, arg2, p)); - unlock_user(p, arg3, ret); - mincore_fail: unlock_user(a, arg1, 0); } return ret; @@ -11956,8 +11942,7 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, ret =3D get_errno(sys_utimensat(arg1, NULL, tsp, arg4)); else { if (!(p =3D lock_user_string(arg2))) { - ret =3D -TARGET_EFAULT; - goto fail; + return -TARGET_EFAULT; } ret =3D get_errno(sys_utimensat(arg1, path(p), tsp, arg4)); unlock_user(p, arg2, 0); @@ -12599,7 +12584,6 @@ static abi_long do_syscall1(void *cpu_env, int num,= abi_long arg1, qemu_log_mask(LOG_UNIMP, "Unsupported syscall: %d\n", num); return -TARGET_ENOSYS; } -fail: return ret; } =20 --=20 2.17.1