From nobody Mon Feb 9 09:09:28 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1619283028; cv=none; d=zohomail.com; s=zohoarc; b=XYKNIdT1Zz8x/Ra1ekn8NbzaAUq+CIw7vsC94zgw0ptAy1dmFwauMQmWh0tXI196DOTlbpyH2UfABqaCfRhGgmsSw3eTSfN7EqTAhFDGUG2aXySoNfT9dNuefkFj7Md3By6KD7DKmSbob6fbUXjDF9esCA6b2R22IZN4N24sl00= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1619283028; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=W0Kb819q+mXdmsjSgnvN1FJvqrihYP2dk/lhl4G+fRg=; b=F3aY0hYy0S10OnsUw4MyjT2Ve8QOVl458Alupr/ptGH38/nMgIicobatCBUudyYqpQe8PguxTrMKfF2WNzY8ULCisy+GPFNEcUxW1vn4SvDoQuk9/kkSm9pIS4g59nxzf0QMMiZq2EvVB1GU/sFkcMjilD28l4doYTOJTCkHda4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1619283028714480.5663791480873; Sat, 24 Apr 2021 09:50:28 -0700 (PDT) Received: from localhost ([::1]:51530 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1laLUA-0000ep-DY for importer@patchew.org; Sat, 24 Apr 2021 12:50:26 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:50788) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1laKip-00064s-FM for qemu-devel@nongnu.org; Sat, 24 Apr 2021 12:01:31 -0400 Received: from mail-il1-x132.google.com ([2607:f8b0:4864:20::132]:39674) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1laKiS-0004Lv-VT for qemu-devel@nongnu.org; Sat, 24 Apr 2021 12:01:31 -0400 Received: by mail-il1-x132.google.com with SMTP id b17so43160962ilh.6 for ; Sat, 24 Apr 2021 09:01:05 -0700 (PDT) Received: from dune.bsdimp.com (50-253-99-174-static.hfc.comcastbusiness.net. [50.253.99.174]) by smtp.gmail.com with ESMTPSA id h4sm4055901ili.52.2021.04.24.09.01.03 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sat, 24 Apr 2021 09:01:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=W0Kb819q+mXdmsjSgnvN1FJvqrihYP2dk/lhl4G+fRg=; b=u+4OawJl5IOpTzkcZhkrDxcNTjjbqzx2VUNqpl7l5pcIRTt567yJV3LfG4E5KMPocS xESBOukOcYKr5/tJ0CxAziSKIWUbq/Mau8UGFnEwWGT05G/mKM4YDsrHS0flW+6TYJI4 2sgNtuTuVV4F6122e9A9o7P2/mj/037qACktoTwymXjeHS1ilEB35NG0jv4+U6f3h6AE ECyVtHlqgoNhKKbZyJuI3xRr2TLkorje6zFmuESuyLxYUki2Ehl/psN+0+iEeioWctq/ hJH7Q+2jlAIBdfxVadLJvcUh0MCCS7Y0nfPeJhXvLi2EPnEPqhFR+TtQ2w1p5THdlEjs OpCg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=W0Kb819q+mXdmsjSgnvN1FJvqrihYP2dk/lhl4G+fRg=; b=siM9DB7C681o4Gd9fg0iWg6ZxPzNWV7txZ9Oe+C9PsM6PYVmITF7JN15UJlGEGmQ7k SizN2pUZ/2jGYNoJ/yPgBwCaWK8EMpkpj8lq8oFJNBj1YL84cyZbshrPz+RBOyNVvmGa Dyl5AXdMLy+Jqb7aWQfNaNOkqrN1Na9KFsSox5tzxlwyhHI5ar9ClyTgnH3MGD2KjdtA hAP93dBCn8X7T5I6u4qeugtjYL32oxVgdCNUwqmb9F13SX0jiUegt3L4oClo4ve+whs5 Da2lKLXJrrQvcvgf93lMzrf9llEFRxe1kdn+WtC2M5ZTYTwh2WhK+OELvMnKKbzSNuPn ALWw== X-Gm-Message-State: AOAM531KPZ1i86p1fUkOYpMunIGhMxoL23yXzeeqcd0NKNI9FOsMIQjs v2rJEHMfTJisD/hc5rrAzbuCcw3wsH6B1Pt0 X-Google-Smtp-Source: ABdhPJy7RESc5b9iknABa+tddrzkwS+gJQH2rnYGC+3ze7X+N+c59qlqzpnGU4Xq5zkmZ9gNOUrLzA== X-Received: by 2002:a92:3644:: with SMTP id d4mr7387092ilf.53.1619280064126; Sat, 24 Apr 2021 09:01:04 -0700 (PDT) From: imp@bsdimp.com To: qemu-devel@nongnu.org Subject: [PATCH v2 45/48] bsd-user: style tweak: Use preferred {} in if/else statements. Date: Sat, 24 Apr 2021 10:00:13 -0600 Message-Id: <20210424160016.15200-46-imp@bsdimp.com> X-Mailer: git-send-email 2.22.1 In-Reply-To: <20210424160016.15200-1-imp@bsdimp.com> References: <20210424160016.15200-1-imp@bsdimp.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: none client-ip=2607:f8b0:4864:20::132; envelope-from=imp@bsdimp.com; helo=mail-il1-x132.google.com X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kevans@freebsd.org, arichardson@freebsd.org, Warner Losh Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @bsdimp-com.20150623.gappssmtp.com) Content-Type: text/plain; charset="utf-8" From: Warner Losh Signed-off-by: Warner Losh Reviewed-by: Richard Henderson --- bsd-user/syscall.c | 120 +++++++++++++++++++++++++++++---------------- 1 file changed, 78 insertions(+), 42 deletions(-) diff --git a/bsd-user/syscall.c b/bsd-user/syscall.c index fbe3b3b2fe..36ffa6a880 100644 --- a/bsd-user/syscall.c +++ b/bsd-user/syscall.c @@ -33,11 +33,12 @@ static abi_ulong target_original_brk; =20 static inline abi_long get_errno(abi_long ret) { - if (ret =3D=3D -1) + if (ret =3D=3D -1) { /* XXX need to translate host -> target errnos here */ return -(errno); - else + } else { return ret; + } } =20 #define target_to_host_bitmask(x, tbl) (x) @@ -59,10 +60,12 @@ static abi_long do_obreak(abi_ulong new_brk) abi_long mapped_addr; int new_alloc_size; =20 - if (!new_brk) + if (!new_brk) { return 0; - if (new_brk < target_original_brk) + } + if (new_brk < target_original_brk) { return -TARGET_EINVAL; + } =20 brk_page =3D HOST_PAGE_ALIGN(target_brk); =20 @@ -79,10 +82,11 @@ static abi_long do_obreak(abi_ulong new_brk) MAP_ANON | MAP_FIXED | MAP_PRIVATE, -1, 0)); =20 - if (!is_error(mapped_addr)) + if (!is_error(mapped_addr)) { target_brk =3D new_brk; - else + } else { return mapped_addr; + } =20 return 0; } @@ -98,35 +102,39 @@ static abi_long do_freebsd_sysarch(CPUX86State *env, i= nt op, abi_ulong parms) #ifdef TARGET_ABI32 case TARGET_FREEBSD_I386_SET_GSBASE: case TARGET_FREEBSD_I386_SET_FSBASE: - if (op =3D=3D TARGET_FREEBSD_I386_SET_GSBASE) + if (op =3D=3D TARGET_FREEBSD_I386_SET_GSBASE) { #else case TARGET_FREEBSD_AMD64_SET_GSBASE: case TARGET_FREEBSD_AMD64_SET_FSBASE: - if (op =3D=3D TARGET_FREEBSD_AMD64_SET_GSBASE) + if (op =3D=3D TARGET_FREEBSD_AMD64_SET_GSBASE) { #endif idx =3D R_GS; - else + } else { idx =3D R_FS; - if (get_user(val, parms, abi_ulong)) + } + if (get_user(val, parms, abi_ulong)) { return -TARGET_EFAULT; + } cpu_x86_load_seg(env, idx, 0); env->segs[idx].base =3D val; break; #ifdef TARGET_ABI32 case TARGET_FREEBSD_I386_GET_GSBASE: case TARGET_FREEBSD_I386_GET_FSBASE: - if (op =3D=3D TARGET_FREEBSD_I386_GET_GSBASE) + if (op =3D=3D TARGET_FREEBSD_I386_GET_GSBASE) { #else case TARGET_FREEBSD_AMD64_GET_GSBASE: case TARGET_FREEBSD_AMD64_GET_FSBASE: - if (op =3D=3D TARGET_FREEBSD_AMD64_GET_GSBASE) + if (op =3D=3D TARGET_FREEBSD_AMD64_GET_GSBASE) { #endif idx =3D R_GS; - else + } else { idx =3D R_FS; + } val =3D env->segs[idx].base; - if (put_user(val, parms, abi_ulong)) + if (put_user(val, parms, abi_ulong)) { return -TARGET_EFAULT; + } break; /* XXX handle the others... */ default: @@ -168,14 +176,17 @@ oidfmt(int *oid, int len, char *fmt, uint32_t *kind) =20 j =3D sizeof(buf); i =3D sysctl(qoid, len + 2, buf, &j, 0, 0); - if (i) + if (i) { return i; + } =20 - if (kind) + if (kind) { *kind =3D *(uint32_t *)buf; + } =20 - if (fmt) + if (fmt) { strcpy(fmt, (char *)(buf + sizeof(uint32_t))); + } return (0); } =20 @@ -231,27 +242,34 @@ static abi_long do_freebsd_sysctl(abi_ulong namep, in= t32_t namelen, int32_t *snamep =3D g_malloc(sizeof(int32_t) * namelen), *p, *q, i; uint32_t kind =3D 0; =20 - if (oldlenp) + if (oldlenp) { get_user_ual(oldlen, oldlenp); - if (!(hnamep =3D lock_user(VERIFY_READ, namep, namelen, 1))) + } + if (!(hnamep =3D lock_user(VERIFY_READ, namep, namelen, 1))) { return -TARGET_EFAULT; - if (newp && !(hnewp =3D lock_user(VERIFY_READ, newp, newlen, 1))) + } + if (newp && !(hnewp =3D lock_user(VERIFY_READ, newp, newlen, 1))) { return -TARGET_EFAULT; - if (!(holdp =3D lock_user(VERIFY_WRITE, oldp, oldlen, 0))) + } + if (!(holdp =3D lock_user(VERIFY_WRITE, oldp, oldlen, 0))) { return -TARGET_EFAULT; + } holdlen =3D oldlen; - for (p =3D hnamep, q =3D snamep, i =3D 0; i < namelen; p++, i++) + for (p =3D hnamep, q =3D snamep, i =3D 0; i < namelen; p++, i++) { *q++ =3D tswap32(*p); + } oidfmt(snamep, namelen, NULL, &kind); /* XXX swap hnewp */ ret =3D get_errno(sysctl(snamep, namelen, holdp, &holdlen, hnewp, newl= en)); - if (!ret) + if (!ret) { sysctl_oldcvt(holdp, holdlen, kind); + } put_user_ual(holdlen, oldlenp); unlock_user(hnamep, namep, 0); unlock_user(holdp, oldp, holdlen); - if (hnewp) + if (hnewp) { unlock_user(hnewp, newp, 0); + } g_free(snamep); return ret; } @@ -271,8 +289,9 @@ static abi_long lock_iovec(int type, struct iovec *vec,= abi_ulong target_addr, =20 target_vec =3D lock_user(VERIFY_READ, target_addr, count * sizeof(struct target_iovec), 1); - if (!target_vec) + if (!target_vec) { return -TARGET_EFAULT; + } for (i =3D 0; i < count; i++) { base =3D tswapl(target_vec[i].iov_base); vec[i].iov_len =3D tswapl(target_vec[i].iov_len); @@ -300,8 +319,9 @@ static abi_long unlock_iovec(struct iovec *vec, abi_ulo= ng target_addr, =20 target_vec =3D lock_user(VERIFY_READ, target_addr, count * sizeof(struct target_iovec), 1); - if (!target_vec) + if (!target_vec) { return -TARGET_EFAULT; + } for (i =3D 0; i < count; i++) { if (target_vec[i].iov_base) { base =3D tswapl(target_vec[i].iov_base); @@ -332,8 +352,9 @@ abi_long do_freebsd_syscall(void *cpu_env, int num, abi= _long arg1, #endif record_syscall_start(cpu, num, arg1, arg2, arg3, arg4, arg5, arg6, 0, = 0); =20 - if (do_strace) + if (do_strace) { print_freebsd_syscall(num, arg1, arg2, arg3, arg4, arg5, arg6); + } =20 switch (num) { case TARGET_FREEBSD_NR_exit: @@ -347,14 +368,16 @@ abi_long do_freebsd_syscall(void *cpu_env, int num, a= bi_long arg1, ret =3D 0; /* avoid warning */ break; case TARGET_FREEBSD_NR_read: - if (!(p =3D lock_user(VERIFY_WRITE, arg2, arg3, 0))) + if (!(p =3D lock_user(VERIFY_WRITE, arg2, arg3, 0))) { goto efault; + } ret =3D get_errno(read(arg1, p, arg3)); unlock_user(p, arg2, ret); break; case TARGET_FREEBSD_NR_write: - if (!(p =3D lock_user(VERIFY_READ, arg2, arg3, 1))) + if (!(p =3D lock_user(VERIFY_READ, arg2, arg3, 1))) { goto efault; + } ret =3D get_errno(write(arg1, p, arg3)); unlock_user(p, arg2, 0); break; @@ -364,15 +387,17 @@ abi_long do_freebsd_syscall(void *cpu_env, int num, a= bi_long arg1, struct iovec *vec; =20 vec =3D alloca(count * sizeof(struct iovec)); - if (lock_iovec(VERIFY_READ, vec, arg2, count, 1) < 0) + if (lock_iovec(VERIFY_READ, vec, arg2, count, 1) < 0) { goto efault; + } ret =3D get_errno(writev(arg1, vec, count)); unlock_iovec(vec, arg2, count, 0); } break; case TARGET_FREEBSD_NR_open: - if (!(p =3D lock_user_string(arg1))) + if (!(p =3D lock_user_string(arg1))) { goto efault; + } ret =3D get_errno(open(path(p), target_to_host_bitmask(arg2, fcntl_flags_tbl), arg3)); @@ -413,8 +438,9 @@ abi_long do_freebsd_syscall(void *cpu_env, int num, abi= _long arg1, #ifdef DEBUG gemu_log(" =3D %ld\n", ret); #endif - if (do_strace) + if (do_strace) { print_freebsd_syscall_ret(num, ret); + } =20 record_syscall_return(cpu, num, ret); return ret; @@ -437,8 +463,9 @@ abi_long do_netbsd_syscall(void *cpu_env, int num, abi_= long arg1, =20 record_syscall_start(cpu, num, arg1, arg2, arg3, arg4, arg5, arg6, 0, = 0); =20 - if (do_strace) + if (do_strace) { print_netbsd_syscall(num, arg1, arg2, arg3, arg4, arg5, arg6); + } =20 switch (num) { case TARGET_NETBSD_NR_exit: @@ -452,20 +479,23 @@ abi_long do_netbsd_syscall(void *cpu_env, int num, ab= i_long arg1, ret =3D 0; /* avoid warning */ break; case TARGET_NETBSD_NR_read: - if (!(p =3D lock_user(VERIFY_WRITE, arg2, arg3, 0))) + if (!(p =3D lock_user(VERIFY_WRITE, arg2, arg3, 0))) { goto efault; + } ret =3D get_errno(read(arg1, p, arg3)); unlock_user(p, arg2, ret); break; case TARGET_NETBSD_NR_write: - if (!(p =3D lock_user(VERIFY_READ, arg2, arg3, 1))) + if (!(p =3D lock_user(VERIFY_READ, arg2, arg3, 1))) { goto efault; + } ret =3D get_errno(write(arg1, p, arg3)); unlock_user(p, arg2, 0); break; case TARGET_NETBSD_NR_open: - if (!(p =3D lock_user_string(arg1))) + if (!(p =3D lock_user_string(arg1))) { goto efault; + } ret =3D get_errno(open(path(p), target_to_host_bitmask(arg2, fcntl_flags_tbl), arg3)); @@ -494,8 +524,9 @@ abi_long do_netbsd_syscall(void *cpu_env, int num, abi_= long arg1, #ifdef DEBUG gemu_log(" =3D %ld\n", ret); #endif - if (do_strace) + if (do_strace) { print_netbsd_syscall_ret(num, ret); + } =20 record_syscall_return(cpu, num, ret); return ret; @@ -518,8 +549,9 @@ abi_long do_openbsd_syscall(void *cpu_env, int num, abi= _long arg1, =20 record_syscall_start(cpu, num, arg1, arg2, arg3, arg4, arg5, arg6, 0, = 0); =20 - if (do_strace) + if (do_strace) { print_openbsd_syscall(num, arg1, arg2, arg3, arg4, arg5, arg6); + } =20 switch (num) { case TARGET_OPENBSD_NR_exit: @@ -533,20 +565,23 @@ abi_long do_openbsd_syscall(void *cpu_env, int num, a= bi_long arg1, ret =3D 0; /* avoid warning */ break; case TARGET_OPENBSD_NR_read: - if (!(p =3D lock_user(VERIFY_WRITE, arg2, arg3, 0))) + if (!(p =3D lock_user(VERIFY_WRITE, arg2, arg3, 0))) { goto efault; + } ret =3D get_errno(read(arg1, p, arg3)); unlock_user(p, arg2, ret); break; case TARGET_OPENBSD_NR_write: - if (!(p =3D lock_user(VERIFY_READ, arg2, arg3, 1))) + if (!(p =3D lock_user(VERIFY_READ, arg2, arg3, 1))) { goto efault; + } ret =3D get_errno(write(arg1, p, arg3)); unlock_user(p, arg2, 0); break; case TARGET_OPENBSD_NR_open: - if (!(p =3D lock_user_string(arg1))) + if (!(p =3D lock_user_string(arg1))) { goto efault; + } ret =3D get_errno(open(path(p), target_to_host_bitmask(arg2, fcntl_flags_tbl), @@ -576,8 +611,9 @@ abi_long do_openbsd_syscall(void *cpu_env, int num, abi= _long arg1, #ifdef DEBUG gemu_log(" =3D %ld\n", ret); #endif - if (do_strace) + if (do_strace) { print_openbsd_syscall_ret(num, ret); + } =20 record_syscall_return(cpu, num, ret); return ret; --=20 2.22.1