From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358912; cv=none; d=zohomail.com; s=zohoarc; b=n4gEe4VCUsDNu1b8o6rajCPAqqkWZkoYBsmR6GfeaQDDq4HX4eU25oRUysRNr7oyjNsmjt3x8hlSB7pC1ugp0HEdRXt5LxIvrDpxsMuOdabSHOevDc5eBWZpN387nmITGMF5ddQSgKn0wMXnruAwZGE+IQtjdQNy/P+/XtWrVgo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358912; h=Content-Type:Content-Transfer-Encoding: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=QTZg7ohzhf5vIkn2M8t+ZauST1BdYT+bIyHoB9OoBQg=; b=aOsIY7Dl5+cF5nMF8d+MXHs/NBWmxp5mUuTFlImMS01wOCUnH1kRLAOByc7/VhVuHmHCJFq+SemSJpWCK6sQ9j4UkygtCjhM3qYq9aMYu44+8qQUSRgsJ+tXYRXQOKpALGzKFY3cqQ19+0sfc0r/Bap8MJD/JHuX7FvwN8UK9tU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1673358912343268.5695993795889; Tue, 10 Jan 2023 05:55:12 -0800 (PST) Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-275-pNt7zjeKM6GqoZMinczXQA-1; Tue, 10 Jan 2023 08:54:32 -0500 Received: from smtp.corp.redhat.com (int-mx10.intmail.prod.int.rdu2.redhat.com [10.11.54.10]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 1F603100F90F; Tue, 10 Jan 2023 13:54:26 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id A52D1492B00; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 765DE19465B1; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id F3FDC1946587 for ; Tue, 10 Jan 2023 13:54:23 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id E418140C2006; Tue, 10 Jan 2023 13:54:23 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id C9F0440C2004; Tue, 10 Jan 2023 13:54:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358911; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=QTZg7ohzhf5vIkn2M8t+ZauST1BdYT+bIyHoB9OoBQg=; b=P6Wo0E4+ntFNL1pOXXhy3nJ7svDJ+jjUghdcxFe0UF8gFtQPHxtE80xdFEMeax34SW4KR6 bPGCzX6lI5XIRb7g+cgmSP+reU08aQf/D7SU5LlBCf+biERLOi/e0nhRp+vpwEvm1AKWbB MsIpxCH31xQhQFGrl/5TDiW8RzHt7rY= X-MC-Unique: pNt7zjeKM6GqoZMinczXQA-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 1/9] remote: remove redundant initialization of args variable Date: Tue, 10 Jan 2023 08:54:15 -0500 Message-Id: <20230110135423.2576907-2-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.10 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358912697100002 Every member of the args variable will be initialized explicitly. A few methods had a redundant call to memset the args which can be removed. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/remote/remote_driver.c | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index 43a714c79a..deeabdb24f 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -6000,7 +6000,6 @@ remoteDomainMigrateBegin3(virDomainPtr domain, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 make_nonnull_domain(&args.dom, domain); @@ -6056,7 +6055,6 @@ remoteDomainMigratePrepare3(virConnectPtr dconn, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 args.cookie_in.cookie_in_val =3D (char *)cookiein; @@ -6125,7 +6123,6 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dconn, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 if (!(netst =3D virNetClientStreamNew(priv->remoteProgram, @@ -6200,7 +6197,6 @@ remoteDomainMigratePerform3(virDomainPtr dom, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 make_nonnull_domain(&args.dom, dom); @@ -6260,7 +6256,6 @@ remoteDomainMigrateFinish3(virConnectPtr dconn, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 args.cookie_in.cookie_in_val =3D (char *)cookiein; @@ -6314,8 +6309,6 @@ remoteDomainMigrateConfirm3(virDomainPtr domain, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); - make_nonnull_domain(&args.dom, domain); args.cookie_in.cookie_in_len =3D cookieinlen; args.cookie_in.cookie_in_val =3D (char *) cookiein; @@ -6908,7 +6901,6 @@ remoteDomainMigrateBegin3Params(virDomainPtr domain, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 make_nonnull_domain(&args.dom, domain); @@ -6971,7 +6963,6 @@ remoteDomainMigratePrepare3Params(virConnectPtr dconn, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 if (virTypedParamsSerialize(params, nparams, @@ -7047,7 +7038,6 @@ remoteDomainMigratePrepareTunnel3Params(virConnectPtr= dconn, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 args.cookie_in.cookie_in_val =3D (char *)cookiein; @@ -7129,7 +7119,6 @@ remoteDomainMigratePerform3Params(virDomainPtr dom, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 make_nonnull_domain(&args.dom, dom); @@ -7195,7 +7184,6 @@ remoteDomainMigrateFinish3Params(virConnectPtr dconn, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); memset(&ret, 0, sizeof(ret)); =20 args.cookie_in.cookie_in_val =3D (char *)cookiein; @@ -7261,8 +7249,6 @@ remoteDomainMigrateConfirm3Params(virDomainPtr domain, =20 remoteDriverLock(priv); =20 - memset(&args, 0, sizeof(args)); - make_nonnull_domain(&args.dom, domain); args.cookie_in.cookie_in_len =3D cookieinlen; args.cookie_in.cookie_in_val =3D (char *) cookiein; @@ -7563,8 +7549,6 @@ remoteConnectGetAllDomainStats(virConnectPtr conn, virDomainStatsRecordPtr elem =3D NULL; virDomainStatsRecordPtr *tmpret =3D NULL; =20 - memset(&args, 0, sizeof(args)); - if (ndoms) { args.doms.doms_val =3D g_new0(remote_nonnull_domain, ndoms); =20 --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358876; cv=none; d=zohomail.com; s=zohoarc; b=UL+M8VI91EZAeWnzaHC4tEdQXCYduoYouhiRFVmc2AI2MXzDCExaKFBsLGlwlTUtuJyJK0mtfJZoRVUlWyUeW2D/FmqU8bQNbaLzLd7PcF1gcy7/WCpjYW9MhszdqKMYQFEyh4NlQKZoucP/ibiI09v7FjMBA6+SbSBUKrvFgFY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358876; h=Content-Type:Content-Transfer-Encoding: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=4rhSl8FkfozZQMYjDIV1JBNPn3qTzMbQNkDcD6hamg0=; b=k8SqVNQ09GyDqMqgmW6HOPkkPAnAd2rCIo8NmkU6iK6WIdxriaVSsNqZ8beHik3flUfL98t/EgurTJm4oiXQW0v7K0YRb4QfzEO7P87ZneJNXups6uUl91YtZAauSnJtOsbXXo8iPDY1KLqlvwa9af1Wpcx1gcQY3G4f6/1oxPM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1673358876791265.04384679684836; Tue, 10 Jan 2023 05:54:36 -0800 (PST) Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-502-ICWc71X_ODmMoeU53D3dyw-1; Tue, 10 Jan 2023 08:54:30 -0500 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.rdu2.redhat.com [10.11.54.7]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 1F7B42823814; Tue, 10 Jan 2023 13:54:26 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id E1CC3140EBF6; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id F41F6194E114; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 2076C1946587 for ; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 14E4B40C2005; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id EE8AC40C2004; Tue, 10 Jan 2023 13:54:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358875; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=4rhSl8FkfozZQMYjDIV1JBNPn3qTzMbQNkDcD6hamg0=; b=UED4VUdmvU1+X8b5YC/AkQOX5qlBLf/M+1X5Z4MmsNwSwi1N0D1c+VlvJrbSm6KexUCZsv oJrIriAGkXNl6GFe/1sNcAWVFeEqDVb7DAi/3Hw1D/oHaet2ln+xSkQ1vXh6FfF9aT0Wqp Nkb44+C4rftQTAzatJyKzFKFSZfd4gg= X-MC-Unique: ICWc71X_ODmMoeU53D3dyw-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 2/9] lxc: fix XDR protocol compliance Date: Tue, 10 Jan 2023 08:54:16 -0500 Message-Id: <20230110135423.2576907-3-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.7 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358878441100001 The RFC spec for XDR does not allow enums to omit their values, they must be explicitly given. Don't rely on this rpcgen language extension. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/lxc/lxc_monitor_protocol.x | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/lxc/lxc_monitor_protocol.x b/src/lxc/lxc_monitor_protocol.x index 5e3d5d9983..d6148412b7 100644 --- a/src/lxc/lxc_monitor_protocol.x +++ b/src/lxc/lxc_monitor_protocol.x @@ -7,9 +7,9 @@ %#include "virxdrdefs.h" =20 enum virLXCMonitorExitStatus { - VIR_LXC_MONITOR_EXIT_STATUS_ERROR, - VIR_LXC_MONITOR_EXIT_STATUS_SHUTDOWN, - VIR_LXC_MONITOR_EXIT_STATUS_REBOOT + VIR_LXC_MONITOR_EXIT_STATUS_ERROR =3D 0, + VIR_LXC_MONITOR_EXIT_STATUS_SHUTDOWN =3D 1, + VIR_LXC_MONITOR_EXIT_STATUS_REBOOT =3D 2 }; =20 struct virLXCMonitorExitEventMsg { --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358877; cv=none; d=zohomail.com; s=zohoarc; b=JjzHJTxxyGzenFYCtkIxmZ5yR23f7YH/1cgsHn7F9cA27aR0oSneoFTCZqUoju7W4v+GIOYXNfPfEQBLH53LWzsHEV3jxv04rLIQlBGZPfy2QAng+Elpe4livKwcfqvoBH2y/cVB+FIeXVp55Rb6DYzgsWo+RjLoQ/4/lpwZdCQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358877; h=Content-Type:Content-Transfer-Encoding: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=nuUqBAEYx1X7Yf2B5QKFxxcCO1lqat3wNtO9FjNawRE=; b=ZmEvP6w2Z1KaerrPC8URJTAC2F1E3LE0/hAREnKn9FlNLWZsd2FrT+HreX/EVB0X6j9IFk/+9WRoyhZay+AtqeLREyLz1o2HlOHFCGDP5sbTT01+ihe6IRIu68aLrDzOYUPmAafwqg32r48h6KeE60zSsrhz8ESryKsZDLjYh6U= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1673358877409582.7618547649132; Tue, 10 Jan 2023 05:54:37 -0800 (PST) Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-371-fxkkpHw8OBeAlX4WUuvnAg-1; Tue, 10 Jan 2023 08:54:33 -0500 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id BAC4A85C71D; Tue, 10 Jan 2023 13:54:27 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id 9AA632166B2A; Tue, 10 Jan 2023 13:54:27 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id BDBE21946594; Tue, 10 Jan 2023 13:54:26 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 439401946587 for ; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 387EA40C2004; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1EC1C40C2064; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358876; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=nuUqBAEYx1X7Yf2B5QKFxxcCO1lqat3wNtO9FjNawRE=; b=GTSIiKqkJOf/c0+o/RijAQVoYx/bPmM9txSJ661BEylhJOcrSCDeDTqPW8jclZLzoDxMqG tTGaV6OhKitFy/T3zJinVcuKClQkJ2LSDEkAef2PIOc6piaA8uPXg5rDgb5L4rrPXg9D4i efWInlRQXXQDTYpUwIBFwqYq/98DiH8= X-MC-Unique: fxkkpHw8OBeAlX4WUuvnAg-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 3/9] logging: remove redundant XDR typedef Date: Tue, 10 Jan 2023 08:54:17 -0500 Message-Id: <20230110135423.2576907-4-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.6 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358878445100003 Every enum/struct/union implicitly includes a typedef in the emitted C code. Furthermore, the syntax used to declare the redundant typedef is not compliant with the XDR spec. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/logging/log_protocol.x | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/logging/log_protocol.x b/src/logging/log_protocol.x index 24340433d6..d5541a999a 100644 --- a/src/logging/log_protocol.x +++ b/src/logging/log_protocol.x @@ -22,13 +22,11 @@ struct virLogManagerProtocolDomain { virLogManagerProtocolUUID uuid; virLogManagerProtocolNonNullString name; }; -typedef struct virLogManagerProtocolDomain virLogManagerProtocolDomain; =20 struct virLogManagerProtocolLogFilePosition { unsigned hyper inode; unsigned hyper offset; }; -typedef struct virLogManagerProtocolLogFilePosition virLogManagerProtocolL= ogFilePosition; =20 enum virLogManagerProtocolDomainOpenLogFileFlags { VIR_LOG_MANAGER_PROTOCOL_DOMAIN_OPEN_LOG_FILE_TRUNCATE =3D 1 --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358877; cv=none; d=zohomail.com; s=zohoarc; b=ViINeOFTtUyO4/tdXFNo/uyfWXlb+5ppdL9tOQA8E0RzcJ6hVMe431bqALlFD8L3koUG7R8ntsAs/3JcUeZuyBxrKKrXvOhmuNiFEyqOBqEy4zdk3Tk958B0kSQ/3ViUf9br6yzFZktB7xE+cYK14reVlVxQh7Yo4PClUlmhK7w= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358877; h=Content-Type:Content-Transfer-Encoding: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=7Mm3M7kpyxw0rGQ64qNndKaqynOguX0nyYC+84CaVio=; b=fhT04EkG/Z8JqdDeeKxNV1yUsiaMk+hcG9J3HaqqL8DVsWvG+lJ3RSM7qZxnCfbHct9E9ypbwtit1irI7Z5P7hnpCSM81J41CXJVybgJGVjNjRxbdQEGF+FXuC5c+1wmvtnk4LvgO7uEUy9OLqJsRGUQGPst4V3gy4zGy1QtWAE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1673358877529442.5278675415665; Tue, 10 Jan 2023 05:54:37 -0800 (PST) Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-463-Z2sd-jADNESAzkQ_uAEvuw-1; Tue, 10 Jan 2023 08:54:33 -0500 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 7AF9385C6E1; Tue, 10 Jan 2023 13:54:27 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id 19173492C3F; Tue, 10 Jan 2023 13:54:27 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id ADC44194F855; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 74F4A1946587 for ; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 5D96240C2007; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id 434D240C2064; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358876; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=7Mm3M7kpyxw0rGQ64qNndKaqynOguX0nyYC+84CaVio=; b=LQ9geRmmBYTH16zkNITqBH/y87ndcAzjE+UaJ3tWk3SGzkCbOR6NekeX+81w/cUgyRIecZ JLnYH2iqIZWebNWYZMQ8H13BdWq4WbelaGNzUywZnvUdwkNYxXVkYtP6JbgdcFlzcR9JDa ERgRaXDJZhamxvQA5ONjl4zPuWDrUno= X-MC-Unique: Z2sd-jADNESAzkQ_uAEvuw-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 4/9] rpc: use VIR_LOCK_GUARD in remote client code Date: Tue, 10 Jan 2023 08:54:18 -0500 Message-Id: <20230110135423.2576907-5-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.9 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358878442100002 Using VIR_LOCK_GUARD helps to simplify the control flow logic. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/rpc/gendispatch.pl | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/src/rpc/gendispatch.pl b/src/rpc/gendispatch.pl index 54d55d91e7..63d2411474 100755 --- a/src/rpc/gendispatch.pl +++ b/src/rpc/gendispatch.pl @@ -1818,11 +1818,10 @@ elsif ($mode eq "client") { } } =20 - print "\n"; if ($structprefix eq "admin") { - print " virObjectLock(priv);\n"; + print " VIR_LOCK_GUARD lock =3D virObjectLockGuard(&priv);\= n"; } else { - print " remoteDriverLock(priv);\n"; + print " VIR_LOCK_GUARD lock =3D remoteDriverLock(priv);\n"; } =20 if ($call->{streamflag} ne "none") { @@ -2020,12 +2019,6 @@ elsif ($mode eq "client") { =20 print join("\n", @free_list); =20 - if ($structprefix eq "admin") { - print " virObjectUnlock(priv);\n"; - } else { - print " remoteDriverUnlock(priv);\n"; - } - print " return rv;\n"; print "}\n"; } --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358877; cv=none; d=zohomail.com; s=zohoarc; b=nas1jpwWSb8fpNQkJ+9aF/8PDafZoNfjYxEJ43o61LjGnGifk03f8+yOeqlZVn/lA11QDVTL0QvuQnTq2ig7yJEzNXzCk4i7EwpxTDZocwK7ZRkbJiD9hXgSI0BGzvF8kr5mO/bX71I1Y/U2u0yKnJM7qEbDjsvvt1SR1/l4Irk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358877; h=Content-Type:Content-Transfer-Encoding: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=pxaMd3KB9tEnP7wzlfdIp8xWhQVxaEi/06ueA7EDq7U=; b=EivrFvwiwkQCcfcUWThLfjF2f/vkwVoAUgP0UjJEuGwQmjt7kgW/CGgvb7Pdsxd8X3+TNX0JOjWq/MHQigIjPNc1mlVV2ssTyEnIzFk0g/ST/dC6hP59u5Yk17S/c7VWnkzoOqwBY9DAPEGqkGQKYv1W+jfAcRoN7OSMsdePO9I= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1673358877686842.711076951653; Tue, 10 Jan 2023 05:54:37 -0800 (PST) Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-369-qj_vgSC0MoqeUCahZfc9bA-1; Tue, 10 Jan 2023 08:54:33 -0500 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id DD1C8282381E; Tue, 10 Jan 2023 13:54:26 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id B7AE1492C3E; Tue, 10 Jan 2023 13:54:26 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 7507D194F4B3; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id A9B36194E114 for ; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 9DBDB40C2006; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6CD3F40C2064; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358875; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=pxaMd3KB9tEnP7wzlfdIp8xWhQVxaEi/06ueA7EDq7U=; b=CqE90wAf5bW5qgSujZsLdkmrMdld+tCih6xWmnzMMX9iJep0dhZiUpMlWL7SeRHOj8Eo/Q DStpUZtBOC4GEz+uz4xsRgwvaBXdpm7m6aMVIQqFhwZXYgDyksa8egIC2plAfhSE/pkuZN 5L9Yc3nK+W+Mpbc+lZ7fOM3EjWmlMTE= X-MC-Unique: qj_vgSC0MoqeUCahZfc9bA-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 5/9] remote: use VIR_LOCK_GUARD in client code Date: Tue, 10 Jan 2023 08:54:19 -0500 Message-Id: <20230110135423.2576907-6-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.9 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358878552100007 Using VIR_LOCK_GUARD enables the 'done' goto label to be eliminated. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/remote/remote_driver.c | 1011 ++++++++++-------------------------- 1 file changed, 288 insertions(+), 723 deletions(-) diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index deeabdb24f..b82e86eff8 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -1362,44 +1362,36 @@ remoteConnectClose(virConnectPtr conn) static const char * remoteConnectGetType(virConnectPtr conn) { - char *rv =3D NULL; remote_connect_get_type_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 /* Cached? */ if (priv->type) { - rv =3D priv->type; - goto done; + return priv->type; } =20 memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_GET_TYPE, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_remote_connect_get_type_ret, (char *) &ret) = =3D=3D -1) - goto done; + return NULL; =20 /* Stash. */ - rv =3D priv->type =3D ret.type; - - done: - remoteDriverUnlock(priv); - return rv; + return priv->type =3D ret.type; } =20 static int remoteConnectIsSecure(virConnectPtr conn) { - int rv =3D -1; struct private_data *priv =3D conn->privateData; remote_connect_is_secure_ret ret; - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_IS_SECURE, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_remote_connect_is_secure_ret, (char *) &ret) = =3D=3D -1) - goto done; + return -1; =20 /* We claim to be secure, if the remote driver * transport itself is secure, and the remote @@ -1409,26 +1401,21 @@ static int remoteConnectIsSecure(virConnectPtr conn) * remote driver is used to connect to a XenD * driver using unencrypted HTTP:/// access */ - rv =3D priv->is_secure && ret.secure ? 1 : 0; - - done: - remoteDriverUnlock(priv); - return rv; + return priv->is_secure && ret.secure ? 1 : 0; } =20 static int remoteConnectIsEncrypted(virConnectPtr conn) { - int rv =3D -1; bool encrypted; struct private_data *priv =3D conn->privateData; remote_connect_is_secure_ret ret; - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_IS_SECURE, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_remote_connect_is_secure_ret, (char *) &ret) = =3D=3D -1) - goto done; + return -1; =20 encrypted =3D virNetClientIsEncrypted(priv->client); =20 @@ -1440,11 +1427,7 @@ static int remoteConnectIsEncrypted(virConnectPtr co= nn) * option, since it will almost always be false, * even if secure (eg UNIX sockets). */ - rv =3D encrypted && ret.secure ? 1 : 0; - - done: - remoteDriverUnlock(priv); - return rv; + return encrypted && ret.secure ? 1 : 0; } =20 static int @@ -1458,8 +1441,7 @@ remoteNodeGetCPUStats(virConnectPtr conn, remote_node_get_cpu_stats_ret ret; size_t i; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.nparams =3D *nparams; args.cpuNum =3D cpuNum; @@ -1471,7 +1453,7 @@ remoteNodeGetCPUStats(virConnectPtr conn, (char *) &args, (xdrproc_t) xdr_remote_node_get_cpu_stats_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 /* Check the length of the returned list carefully. */ if (ret.params.params_len > REMOTE_NODE_CPU_STATS_MAX || @@ -1507,8 +1489,6 @@ remoteNodeGetCPUStats(virConnectPtr conn, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_node_get_cpu_stats_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1524,8 +1504,7 @@ remoteNodeGetMemoryStats(virConnectPtr conn, remote_node_get_memory_stats_ret ret; size_t i; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.nparams =3D *nparams; args.cellNum =3D cellNum; @@ -1535,7 +1514,7 @@ remoteNodeGetMemoryStats(virConnectPtr conn, if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_MEMORY_STATS, (xdrproc_t) xdr_remote_node_get_memory_stats_args, (char *) &= args, (xdrproc_t) xdr_remote_node_get_memory_stats_ret, (char *) &r= et) =3D=3D -1) - goto done; + return -1; =20 /* Check the length of the returned list carefully. */ if (ret.params.params_len > REMOTE_NODE_MEMORY_STATS_MAX || @@ -1571,8 +1550,6 @@ remoteNodeGetMemoryStats(virConnectPtr conn, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_node_get_memory_stats_ret, (char *) &r= et); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1582,19 +1559,17 @@ remoteNodeGetCellsFreeMemory(virConnectPtr conn, int startCell, int maxCells) { - int rv =3D -1; remote_node_get_cells_free_memory_args args; remote_node_get_cells_free_memory_ret ret; size_t i; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (maxCells > REMOTE_NODE_MAX_CELLS) { virReportError(VIR_ERR_RPC, _("too many NUMA cells: %d > %d"), maxCells, REMOTE_NODE_MAX_CELLS); - goto done; + return -1; } =20 args.startCell =3D startCell; @@ -1604,18 +1579,14 @@ remoteNodeGetCellsFreeMemory(virConnectPtr conn, if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_CELLS_FREE_MEMORY, (xdrproc_t) xdr_remote_node_get_cells_free_memory_args, (char= *)&args, (xdrproc_t) xdr_remote_node_get_cells_free_memory_ret, (char = *)&ret) =3D=3D -1) - goto done; + return -1; =20 for (i =3D 0; i < ret.cells.cells_len; i++) freeMems[i] =3D ret.cells.cells_val[i]; =20 xdr_free((xdrproc_t) xdr_remote_node_get_cells_free_memory_ret, (char = *) &ret); =20 - rv =3D ret.cells.cells_len; - - done: - remoteDriverUnlock(priv); - return rv; + return ret.cells.cells_len; } =20 static int @@ -1626,14 +1597,13 @@ remoteConnectListDomains(virConnectPtr conn, int *i= ds, int maxids) remote_connect_list_domains_args args; remote_connect_list_domains_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (maxids > REMOTE_DOMAIN_LIST_MAX) { virReportError(VIR_ERR_RPC, _("Too many domains '%d' for limit '%d'"), maxids, REMOTE_DOMAIN_LIST_MAX); - goto done; + return -1; } args.maxids =3D maxids; =20 @@ -1641,7 +1611,7 @@ remoteConnectListDomains(virConnectPtr conn, int *ids= , int maxids) if (call(conn, priv, 0, REMOTE_PROC_CONNECT_LIST_DOMAINS, (xdrproc_t) xdr_remote_connect_list_domains_args, (char *) &a= rgs, (xdrproc_t) xdr_remote_connect_list_domains_ret, (char *) &re= t) =3D=3D -1) - goto done; + return -1; =20 if (ret.ids.ids_len > maxids) { virReportError(VIR_ERR_RPC, @@ -1657,9 +1627,6 @@ remoteConnectListDomains(virConnectPtr conn, int *ids= , int maxids) =20 cleanup: xdr_free((xdrproc_t) xdr_remote_connect_list_domains_ret, (char *) &re= t); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1704,8 +1671,7 @@ remoteDomainBlockStatsFlags(virDomainPtr domain, remote_domain_block_stats_flags_args args; remote_domain_block_stats_flags_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.nparams =3D *nparams; @@ -1716,7 +1682,7 @@ remoteDomainBlockStatsFlags(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_BLOCK_STATS_FLAGS, (xdrproc_t) xdr_remote_domain_block_stats_flags_args, (char *= ) &args, (xdrproc_t) xdr_remote_domain_block_stats_flags_ret, (char *)= &ret) =3D=3D -1) - goto done; + return -1; =20 /* Check the length of the returned list carefully. */ if (ret.params.params_len > REMOTE_DOMAIN_BLOCK_STATS_PARAMETERS_MAX || @@ -1751,8 +1717,6 @@ remoteDomainBlockStatsFlags(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_block_stats_flags_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1765,8 +1729,7 @@ remoteDomainGetMemoryParameters(virDomainPtr domain, remote_domain_get_memory_parameters_args args; remote_domain_get_memory_parameters_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.nparams =3D *nparams; @@ -1776,7 +1739,7 @@ remoteDomainGetMemoryParameters(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_MEMORY_PARAMETE= RS, (xdrproc_t) xdr_remote_domain_get_memory_parameters_args, (ch= ar *) &args, (xdrproc_t) xdr_remote_domain_get_memory_parameters_ret, (cha= r *) &ret) =3D=3D -1) - goto done; + return -1; =20 /* Handle the case when the caller does not know the number of paramet= ers * and is asking for the number of parameters supported @@ -1799,8 +1762,6 @@ remoteDomainGetMemoryParameters(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_memory_parameters_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1813,8 +1774,7 @@ remoteDomainGetNumaParameters(virDomainPtr domain, remote_domain_get_numa_parameters_args args; remote_domain_get_numa_parameters_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.nparams =3D *nparams; @@ -1824,7 +1784,7 @@ remoteDomainGetNumaParameters(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_NUMA_PARAMETERS, (xdrproc_t) xdr_remote_domain_get_numa_parameters_args, (char= *) &args, (xdrproc_t) xdr_remote_domain_get_numa_parameters_ret, (char = *) &ret) =3D=3D -1) - goto done; + return -1; =20 /* Handle the case when the caller does not know the number of paramet= ers * and is asking for the number of parameters supported @@ -1847,8 +1807,6 @@ remoteDomainGetNumaParameters(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_numa_parameters_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1862,8 +1820,7 @@ remoteDomainGetLaunchSecurityInfo(virDomainPtr domain, remote_domain_get_launch_security_info_args args; remote_domain_get_launch_security_info_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; @@ -1872,7 +1829,7 @@ remoteDomainGetLaunchSecurityInfo(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_LAUNCH_SECURITY= _INFO, (xdrproc_t) xdr_remote_domain_get_launch_security_info_args, = (char *) &args, (xdrproc_t) xdr_remote_domain_get_launch_security_info_ret, (= char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -1886,8 +1843,6 @@ remoteDomainGetLaunchSecurityInfo(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_launch_security_info_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1901,8 +1856,7 @@ remoteDomainGetPerfEvents(virDomainPtr domain, remote_domain_get_perf_events_args args; remote_domain_get_perf_events_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; @@ -1911,7 +1865,7 @@ remoteDomainGetPerfEvents(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_PERF_EVENTS, (xdrproc_t) xdr_remote_domain_get_perf_events_args, (char *) = &args, (xdrproc_t) xdr_remote_domain_get_perf_events_ret, (char *) &= ret) =3D=3D -1) - goto done; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -1925,8 +1879,6 @@ remoteDomainGetPerfEvents(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_perf_events_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1939,8 +1891,7 @@ remoteDomainGetBlkioParameters(virDomainPtr domain, remote_domain_get_blkio_parameters_args args; remote_domain_get_blkio_parameters_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.nparams =3D *nparams; @@ -1950,7 +1901,7 @@ remoteDomainGetBlkioParameters(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_BLKIO_PARAMETER= S, (xdrproc_t) xdr_remote_domain_get_blkio_parameters_args, (cha= r *) &args, (xdrproc_t) xdr_remote_domain_get_blkio_parameters_ret, (char= *) &ret) =3D=3D -1) - goto done; + return -1; =20 /* Handle the case when the caller does not know the number of paramet= ers * and is asking for the number of parameters supported @@ -1973,8 +1924,6 @@ remoteDomainGetBlkioParameters(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_blkio_parameters_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -1990,14 +1939,13 @@ remoteDomainGetVcpuPinInfo(virDomainPtr domain, remote_domain_get_vcpu_pin_info_args args; remote_domain_get_vcpu_pin_info_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (ncpumaps > REMOTE_VCPUINFO_MAX) { virReportError(VIR_ERR_RPC, _("vCPU count exceeds maximum: %d > %d"), ncpumaps, REMOTE_VCPUINFO_MAX); - goto done; + return -1; } =20 if (VIR_INT_MULTIPLY_OVERFLOW(ncpumaps, maplen) || @@ -2005,7 +1953,7 @@ remoteDomainGetVcpuPinInfo(virDomainPtr domain, virReportError(VIR_ERR_RPC, _("vCPU map buffer length exceeds maximum: %d > %d"= ), ncpumaps * maplen, REMOTE_CPUMAPS_MAX); - goto done; + return -1; } =20 make_nonnull_domain(&args.dom, domain); @@ -2020,7 +1968,7 @@ remoteDomainGetVcpuPinInfo(virDomainPtr domain, (char *) &args, (xdrproc_t) xdr_remote_domain_get_vcpu_pin_info_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.num > ncpumaps) { virReportError(VIR_ERR_RPC, @@ -2045,9 +1993,6 @@ remoteDomainGetVcpuPinInfo(virDomainPtr domain, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_vcpu_pin_info_ret, (char *)= &ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2057,17 +2002,15 @@ remoteDomainPinEmulator(virDomainPtr dom, int cpumaplen, unsigned int flags) { - int rv =3D -1; struct private_data *priv =3D dom->conn->privateData; remote_domain_pin_emulator_args args; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (cpumaplen > REMOTE_CPUMAP_MAX) { virReportError(VIR_ERR_RPC, _("%s length greater than maximum: %d > %d"), "cpumap", cpumaplen, REMOTE_CPUMAP_MAX); - goto done; + return -1; } =20 make_nonnull_domain(&args.dom, dom); @@ -2078,15 +2021,10 @@ remoteDomainPinEmulator(virDomainPtr dom, if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_PIN_EMULATOR, (xdrproc_t) xdr_remote_domain_pin_emulator_args, (char *) &args, - (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) { - goto done; - } - - rv =3D 0; + (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -2101,15 +2039,14 @@ remoteDomainGetEmulatorPinInfo(virDomainPtr domain, remote_domain_get_emulator_pin_info_args args; remote_domain_get_emulator_pin_info_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 /* There is only one cpumap for all emulator threads */ if (maplen > REMOTE_CPUMAPS_MAX) { virReportError(VIR_ERR_RPC, _("vCPU map buffer length exceeds maximum: %d > %d"= ), maplen, REMOTE_CPUMAPS_MAX); - goto done; + return -1; } =20 make_nonnull_domain(&args.dom, domain); @@ -2123,7 +2060,7 @@ remoteDomainGetEmulatorPinInfo(virDomainPtr domain, (char *) &args, (xdrproc_t) xdr_remote_domain_get_emulator_pin_info_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.cpumaps.cpumaps_len > maplen) { virReportError(VIR_ERR_RPC, @@ -2142,9 +2079,6 @@ remoteDomainGetEmulatorPinInfo(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_emulator_pin_info_ret, (char *) &ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2160,21 +2094,20 @@ remoteDomainGetVcpus(virDomainPtr domain, remote_domain_get_vcpus_args args; remote_domain_get_vcpus_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (maxinfo > REMOTE_VCPUINFO_MAX) { virReportError(VIR_ERR_RPC, _("vCPU count exceeds maximum: %d > %d"), maxinfo, REMOTE_VCPUINFO_MAX); - goto done; + return -1; } if (VIR_INT_MULTIPLY_OVERFLOW(maxinfo, maplen) || maxinfo * maplen > REMOTE_CPUMAPS_MAX) { virReportError(VIR_ERR_RPC, _("vCPU map buffer length exceeds maximum: %d > %d"= ), maxinfo * maplen, REMOTE_CPUMAPS_MAX); - goto done; + return -1; } =20 make_nonnull_domain(&args.dom, domain); @@ -2185,7 +2118,7 @@ remoteDomainGetVcpus(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_VCPUS, (xdrproc_t) xdr_remote_domain_get_vcpus_args, (char *) &args, (xdrproc_t) xdr_remote_domain_get_vcpus_ret, (char *) &ret) = =3D=3D -1) - goto done; + return -1; =20 if (ret.info.info_len > maxinfo) { virReportError(VIR_ERR_RPC, @@ -2217,9 +2150,6 @@ remoteDomainGetVcpus(virDomainPtr domain, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_vcpus_ret, (char *) &ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2234,8 +2164,7 @@ remoteDomainGetIOThreadInfo(virDomainPtr dom, remote_domain_get_iothread_info_args args; remote_domain_get_iothread_info_ret ret; remote_domain_iothread_info *src; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); =20 @@ -2248,7 +2177,7 @@ remoteDomainGetIOThreadInfo(virDomainPtr dom, (char *)&args, (xdrproc_t)xdr_remote_domain_get_iothread_info_ret, (char *)&ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.info.info_len > REMOTE_IOTHREAD_INFO_MAX) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -2287,9 +2216,6 @@ remoteDomainGetIOThreadInfo(virDomainPtr dom, cleanup: xdr_free((xdrproc_t)xdr_remote_domain_get_iothread_info_ret, (char *) &ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2300,8 +2226,7 @@ remoteDomainGetSecurityLabel(virDomainPtr domain, vir= SecurityLabelPtr seclabel) remote_domain_get_security_label_ret ret; struct private_data *priv =3D domain->conn->privateData; int rv =3D -1; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); memset(&ret, 0, sizeof(ret)); @@ -2309,9 +2234,8 @@ remoteDomainGetSecurityLabel(virDomainPtr domain, vir= SecurityLabelPtr seclabel) =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL, (xdrproc_t) xdr_remote_domain_get_security_label_args, (char = *)&args, - (xdrproc_t) xdr_remote_domain_get_security_label_ret, (char *= )&ret) =3D=3D -1) { - goto done; - } + (xdrproc_t) xdr_remote_domain_get_security_label_ret, (char *= )&ret) =3D=3D -1) + return -1; =20 if (ret.label.label_val !=3D NULL) { if (virStrcpyStatic(seclabel->label, ret.label.label_val) < 0) { @@ -2326,9 +2250,6 @@ remoteDomainGetSecurityLabel(virDomainPtr domain, vir= SecurityLabelPtr seclabel) =20 cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_security_label_ret, (char *= )&ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2340,17 +2261,15 @@ remoteDomainGetSecurityLabelList(virDomainPtr domai= n, virSecurityLabelPtr* secla struct private_data *priv =3D domain->conn->privateData; size_t i; int rv =3D -1; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); memset(&ret, 0, sizeof(ret)); =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL_= LIST, (xdrproc_t) xdr_remote_domain_get_security_label_list_args, (= char *)&args, - (xdrproc_t) xdr_remote_domain_get_security_label_list_ret, (c= har *)&ret) =3D=3D -1) { - goto done; - } + (xdrproc_t) xdr_remote_domain_get_security_label_list_ret, (c= har *)&ret) =3D=3D -1) + return -1; =20 *seclabels =3D g_new0(virSecurityLabel, ret.labels.labels_len); =20 @@ -2370,9 +2289,6 @@ remoteDomainGetSecurityLabelList(virDomainPtr domain,= virSecurityLabelPtr* secla =20 cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_security_label_list_ret, (c= har *)&ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2382,12 +2298,10 @@ remoteDomainGetState(virDomainPtr domain, int *reason, unsigned int flags) { - int rv =3D -1; remote_domain_get_state_args args; remote_domain_get_state_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; @@ -2396,17 +2310,13 @@ remoteDomainGetState(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_STATE, (xdrproc_t) xdr_remote_domain_get_state_args, (char *) &args, (xdrproc_t) xdr_remote_domain_get_state_ret, (char *) &ret) = =3D=3D -1) - goto done; + return -1; =20 *state =3D ret.state; if (reason) *reason =3D ret.reason; =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static int @@ -2415,17 +2325,15 @@ remoteNodeGetSecurityModel(virConnectPtr conn, virS= ecurityModelPtr secmodel) remote_node_get_security_model_ret ret; struct private_data *priv =3D conn->privateData; int rv =3D -1; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); memset(secmodel, 0, sizeof(*secmodel)); =20 if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_SECURITY_MODEL, (xdrproc_t) xdr_void, NULL, - (xdrproc_t) xdr_remote_node_get_security_model_ret, (char *)&= ret) =3D=3D -1) { - goto done; - } + (xdrproc_t) xdr_remote_node_get_security_model_ret, (char *)&= ret) =3D=3D -1) + return -1; =20 if (ret.model.model_val !=3D NULL) { if (virStrcpyStatic(secmodel->model, ret.model.model_val) < 0) { @@ -2447,9 +2355,6 @@ remoteNodeGetSecurityModel(virConnectPtr conn, virSec= urityModelPtr secmodel) =20 cleanup: xdr_free((xdrproc_t) xdr_remote_node_get_security_model_ret, (char *)&= ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2460,12 +2365,10 @@ remoteDomainMigratePrepare(virConnectPtr dconn, unsigned long flags, const char *dname, unsigned long resource) { - int rv =3D -1; remote_domain_migrate_prepare_args args; remote_domain_migrate_prepare_ret ret; struct private_data *priv =3D dconn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.uri_in =3D uri_in =3D=3D NULL ? NULL : (char **) &uri_in; args.flags =3D flags; @@ -2476,7 +2379,7 @@ remoteDomainMigratePrepare(virConnectPtr dconn, if (call(dconn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE, (xdrproc_t) xdr_remote_domain_migrate_prepare_args, (char *) = &args, (xdrproc_t) xdr_remote_domain_migrate_prepare_ret, (char *) &= ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.cookie.cookie_len > 0) { *cookie =3D ret.cookie.cookie_val; /* Caller frees. */ @@ -2486,11 +2389,7 @@ remoteDomainMigratePrepare(virConnectPtr dconn, *uri_out =3D *ret.uri_out; /* Caller frees. */ =20 VIR_FREE(ret.uri_out); - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static int @@ -2505,8 +2404,7 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, remote_domain_migrate_prepare2_args args; remote_domain_migrate_prepare2_ret ret; struct private_data *priv =3D dconn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.uri_in =3D uri_in =3D=3D NULL ? NULL : (char **) &uri_in; args.flags =3D flags; @@ -2518,7 +2416,7 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, if (call(dconn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE2, (xdrproc_t) xdr_remote_domain_migrate_prepare2_args, (char *)= &args, (xdrproc_t) xdr_remote_domain_migrate_prepare2_ret, (char *) = &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.cookie.cookie_len > 0) { if (!cookie || !cookielen) { @@ -2542,7 +2440,6 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, =20 done: VIR_FREE(ret.uri_out); - remoteDriverUnlock(priv); return rv; error: if (ret.cookie.cookie_len) @@ -2555,20 +2452,18 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, static int remoteDomainCreate(virDomainPtr domain) { - int rv =3D -1; remote_domain_create_args args; remote_domain_lookup_by_uuid_args args2; remote_domain_lookup_by_uuid_ret ret2; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_CREATE, (xdrproc_t) xdr_remote_domain_create_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; =20 /* Need to do a lookup figure out ID of newly started guest, because * bug in design of REMOTE_PROC_DOMAIN_CREATE means we aren't getting @@ -2579,27 +2474,21 @@ remoteDomainCreate(virDomainPtr domain) if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_LOOKUP_BY_UUID, (xdrproc_t) xdr_remote_domain_lookup_by_uuid_args, (char *) &= args2, (xdrproc_t) xdr_remote_domain_lookup_by_uuid_ret, (char *) &r= et2) =3D=3D -1) - goto done; + return -1; =20 domain->id =3D ret2.dom.id; xdr_free((xdrproc_t) &xdr_remote_domain_lookup_by_uuid_ret, (char *) &= ret2); =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static char * remoteDomainGetSchedulerType(virDomainPtr domain, int *nparams) { - char *rv =3D NULL; remote_domain_get_scheduler_type_args args; remote_domain_get_scheduler_type_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); =20 @@ -2607,16 +2496,12 @@ remoteDomainGetSchedulerType(virDomainPtr domain, i= nt *nparams) if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_SCHEDULER_TYPE, (xdrproc_t) xdr_remote_domain_get_scheduler_type_args, (char = *) &args, (xdrproc_t) xdr_remote_domain_get_scheduler_type_ret, (char *= ) &ret) =3D=3D -1) - goto done; + return NULL; =20 if (nparams) *nparams =3D ret.nparams; =20 /* Caller frees this. */ - rv =3D ret.type; - - done: - remoteDriverUnlock(priv); - return rv; + return ret.type; } =20 static int @@ -2630,15 +2515,14 @@ remoteDomainMemoryStats(virDomainPtr domain, remote_domain_memory_stats_ret ret; struct private_data *priv =3D domain->conn->privateData; size_t i; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); if (nr_stats > REMOTE_DOMAIN_MEMORY_STATS_MAX) { virReportError(VIR_ERR_RPC, _("too many memory stats requested: %d > %d"), nr_s= tats, REMOTE_DOMAIN_MEMORY_STATS_MAX); - goto done; + return -1; } args.maxStats =3D nr_stats; args.flags =3D flags; @@ -2649,7 +2533,7 @@ remoteDomainMemoryStats(virDomainPtr domain, (char *) &args, (xdrproc_t) xdr_remote_domain_memory_stats_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 for (i =3D 0; i < ret.stats.stats_len; i++) { stats[i].tag =3D ret.stats.stats_val[i].tag; @@ -2658,8 +2542,6 @@ remoteDomainMemoryStats(virDomainPtr domain, rv =3D ret.stats.stats_len; xdr_free((xdrproc_t) xdr_remote_domain_memory_stats_ret, (char *) &ret= ); =20 - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2675,14 +2557,13 @@ remoteDomainBlockPeek(virDomainPtr domain, remote_domain_block_peek_args args; remote_domain_block_peek_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (size > REMOTE_DOMAIN_BLOCK_PEEK_BUFFER_MAX) { virReportError(VIR_ERR_RPC, _("block peek request too large for remote protocol= , %zi > %d"), size, REMOTE_DOMAIN_BLOCK_PEEK_BUFFER_MAX); - goto done; + return -1; } =20 make_nonnull_domain(&args.dom, domain); @@ -2697,7 +2578,7 @@ remoteDomainBlockPeek(virDomainPtr domain, (char *) &args, (xdrproc_t) xdr_remote_domain_block_peek_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.buffer.buffer_len !=3D size) { virReportError(VIR_ERR_RPC, "%s", @@ -2710,9 +2591,6 @@ remoteDomainBlockPeek(virDomainPtr domain, =20 cleanup: VIR_FREE(ret.buffer.buffer_val); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2727,14 +2605,13 @@ remoteDomainMemoryPeek(virDomainPtr domain, remote_domain_memory_peek_args args; remote_domain_memory_peek_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (size > REMOTE_DOMAIN_MEMORY_PEEK_BUFFER_MAX) { virReportError(VIR_ERR_RPC, _("memory peek request too large for remote protoco= l, %zi > %d"), size, REMOTE_DOMAIN_MEMORY_PEEK_BUFFER_MAX); - goto done; + return -1; } =20 make_nonnull_domain(&args.dom, domain); @@ -2748,7 +2625,7 @@ remoteDomainMemoryPeek(virDomainPtr domain, (char *) &args, (xdrproc_t) xdr_remote_domain_memory_peek_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.buffer.buffer_len !=3D size) { virReportError(VIR_ERR_RPC, "%s", @@ -2762,8 +2639,6 @@ remoteDomainMemoryPeek(virDomainPtr domain, cleanup: VIR_FREE(ret.buffer.buffer_val); =20 - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2772,12 +2647,10 @@ static int remoteDomainGetBlockJobInfo(virDomainPtr= domain, virDomainBlockJobInfoPtr info, unsigned int flags) { - int rv =3D -1; remote_domain_get_block_job_info_args args; remote_domain_get_block_job_info_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.path =3D (char *)path; @@ -2788,21 +2661,17 @@ static int remoteDomainGetBlockJobInfo(virDomainPtr= domain, (char *)&args, (xdrproc_t)xdr_remote_domain_get_block_job_info_ret, (char *)&ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.found) { info->type =3D ret.type; info->bandwidth =3D ret.bandwidth; info->cur =3D ret.cur; info->end =3D ret.end; - rv =3D 1; + return 1; } else { - rv =3D 0; + return 0; } - - done: - remoteDriverUnlock(priv); - return rv; } =20 static int remoteDomainGetBlockIoTune(virDomainPtr domain, @@ -2815,8 +2684,7 @@ static int remoteDomainGetBlockIoTune(virDomainPtr do= main, remote_domain_get_block_io_tune_args args; remote_domain_get_block_io_tune_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.disk =3D disk ? (char **)&disk : NULL; @@ -2831,7 +2699,7 @@ static int remoteDomainGetBlockIoTune(virDomainPtr do= main, (char *) &args, (xdrproc_t) xdr_remote_domain_get_block_io_tune_ret, (char *) &ret) =3D=3D -1) { - goto done; + return -1; } =20 /* Handle the case when the caller does not know the number of paramet= ers @@ -2855,8 +2723,6 @@ static int remoteDomainGetBlockIoTune(virDomainPtr do= main, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_block_io_tune_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2872,20 +2738,19 @@ static int remoteDomainGetCPUStats(virDomainPtr dom= ain, remote_domain_get_cpu_stats_ret ret; int rv =3D -1; int cpu; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (nparams > REMOTE_NODE_CPU_STATS_MAX) { virReportError(VIR_ERR_RPC, _("nparams count exceeds maximum: %u > %u"), nparams, REMOTE_NODE_CPU_STATS_MAX); - goto done; + return -1; } if (ncpus > REMOTE_DOMAIN_GET_CPU_STATS_NCPUS_MAX) { virReportError(VIR_ERR_RPC, _("ncpus count exceeds maximum: %u > %u"), ncpus, REMOTE_DOMAIN_GET_CPU_STATS_NCPUS_MAX); - goto done; + return -1; } =20 make_nonnull_domain(&args.dom, domain); @@ -2901,7 +2766,7 @@ static int remoteDomainGetCPUStats(virDomainPtr domai= n, (char *) &args, (xdrproc_t) xdr_remote_domain_get_cpu_stats_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 /* Check the length of the returned list carefully. */ if (ret.params.params_len > nparams * ncpus || @@ -2947,8 +2812,6 @@ static int remoteDomainGetCPUStats(virDomainPtr domai= n, =20 xdr_free((xdrproc_t) xdr_remote_domain_get_cpu_stats_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -2961,21 +2824,19 @@ remoteConnectNetworkEventRegisterAny(virConnectPtr = conn, void *opaque, virFreeCallback freecb) { - int rv =3D -1; struct private_data *priv =3D conn->privateData; remote_connect_network_event_register_any_args args; remote_connect_network_event_register_any_ret ret; int callbackID; int count; remote_nonnull_network network; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((count =3D virNetworkEventStateRegisterClient(conn, priv->eventSta= te, net, eventID, callback, opaque, freecb, &callbackID)) < 0) - goto done; + return -1; =20 /* If this is the first callback for this eventID, we need to enable * events on the server */ @@ -2994,17 +2855,13 @@ remoteConnectNetworkEventRegisterAny(virConnectPtr = conn, (xdrproc_t) xdr_remote_connect_network_event_register_any= _ret, (char *) &ret) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } virObjectEventStateSetRemote(conn, priv->eventState, callbackID, ret.callbackID); } =20 - rv =3D callbackID; - - done: - remoteDriverUnlock(priv); - return rv; + return callbackID; } =20 =20 @@ -3013,21 +2870,19 @@ remoteConnectNetworkEventDeregisterAny(virConnectPt= r conn, int callbackID) { struct private_data *priv =3D conn->privateData; - int rv =3D -1; remote_connect_network_event_deregister_any_args args; int eventID; int remoteID; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((eventID =3D virObjectEventStateEventID(conn, priv->eventState, callbackID, &remoteID)) < 0) - goto done; + return -1; =20 if ((count =3D virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, true)) < 0) - goto done; + return -1; =20 /* If that was the last callback for this eventID, we need to disable * events on the server */ @@ -3037,14 +2892,10 @@ remoteConnectNetworkEventDeregisterAny(virConnectPt= r conn, if (call(conn, priv, 0, REMOTE_PROC_CONNECT_NETWORK_EVENT_DEREGIST= ER_ANY, (xdrproc_t) xdr_remote_connect_network_event_deregister_a= ny_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static int @@ -3055,21 +2906,19 @@ remoteConnectStoragePoolEventRegisterAny(virConnect= Ptr conn, void *opaque, virFreeCallback freecb) { - int rv =3D -1; struct private_data *priv =3D conn->privateData; remote_connect_storage_pool_event_register_any_args args; remote_connect_storage_pool_event_register_any_ret ret; int callbackID; int count; remote_nonnull_storage_pool storage_pool; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((count =3D virStoragePoolEventStateRegisterClient(conn, priv->even= tState, pool, eventID, cal= lback, opaque, freecb, &callbackID)) < 0) - goto done; + return -1; =20 /* If this is the first callback for this eventID, we need to enable * events on the server */ @@ -3088,18 +2937,14 @@ remoteConnectStoragePoolEventRegisterAny(virConnect= Ptr conn, (xdrproc_t) xdr_remote_connect_storage_pool_event_registe= r_any_ret, (char *) &ret) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } =20 virObjectEventStateSetRemote(conn, priv->eventState, callbackID, ret.callbackID); } =20 - rv =3D callbackID; - - done: - remoteDriverUnlock(priv); - return rv; + return callbackID; } =20 static int @@ -3107,21 +2952,19 @@ remoteConnectStoragePoolEventDeregisterAny(virConne= ctPtr conn, int callbackID) { struct private_data *priv =3D conn->privateData; - int rv =3D -1; remote_connect_storage_pool_event_deregister_any_args args; int eventID; int remoteID; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((eventID =3D virObjectEventStateEventID(conn, priv->eventState, callbackID, &remoteID)) < 0) - goto done; + return -1; =20 if ((count =3D virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, true)) < 0) - goto done; + return -1; =20 /* If that was the last callback for this eventID, we need to disable * events on the server */ @@ -3131,15 +2974,11 @@ remoteConnectStoragePoolEventDeregisterAny(virConne= ctPtr conn, if (call(conn, priv, 0, REMOTE_PROC_CONNECT_STORAGE_POOL_EVENT_DER= EGISTER_ANY, (xdrproc_t) xdr_remote_connect_storage_pool_event_deregis= ter_any_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; =20 } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -3151,21 +2990,19 @@ remoteConnectNodeDeviceEventRegisterAny(virConnectP= tr conn, void *opaque, virFreeCallback freecb) { - int rv =3D -1; struct private_data *priv =3D conn->privateData; remote_connect_node_device_event_register_any_args args; remote_connect_node_device_event_register_any_ret ret; int callbackID; int count; remote_nonnull_node_device node_device; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((count =3D virNodeDeviceEventStateRegisterClient(conn, priv->event= State, dev, eventID, callb= ack, opaque, freecb, &callbackID)) < 0) - goto done; + return -1; =20 /* If this is the first callback for this eventID, we need to enable * events on the server */ @@ -3184,18 +3021,14 @@ remoteConnectNodeDeviceEventRegisterAny(virConnectP= tr conn, (xdrproc_t) xdr_remote_connect_node_device_event_register= _any_ret, (char *) &ret) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } =20 virObjectEventStateSetRemote(conn, priv->eventState, callbackID, ret.callbackID); } =20 - rv =3D callbackID; - - done: - remoteDriverUnlock(priv); - return rv; + return callbackID; } =20 =20 @@ -3204,21 +3037,19 @@ remoteConnectNodeDeviceEventDeregisterAny(virConnec= tPtr conn, int callbackID) { struct private_data *priv =3D conn->privateData; - int rv =3D -1; remote_connect_node_device_event_deregister_any_args args; int eventID; int remoteID; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((eventID =3D virObjectEventStateEventID(conn, priv->eventState, callbackID, &remoteID)) < 0) - goto done; + return -1; =20 if ((count =3D virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, true)) < 0) - goto done; + return -1; =20 /* If that was the last callback for this eventID, we need to disable * events on the server */ @@ -3228,15 +3059,11 @@ remoteConnectNodeDeviceEventDeregisterAny(virConnec= tPtr conn, if (call(conn, priv, 0, REMOTE_PROC_CONNECT_NODE_DEVICE_EVENT_DERE= GISTER_ANY, (xdrproc_t) xdr_remote_connect_node_device_event_deregist= er_any_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; =20 } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -3248,21 +3075,19 @@ remoteConnectSecretEventRegisterAny(virConnectPtr c= onn, void *opaque, virFreeCallback freecb) { - int rv =3D -1; struct private_data *priv =3D conn->privateData; remote_connect_secret_event_register_any_args args; remote_connect_secret_event_register_any_ret ret; int callbackID; int count; remote_nonnull_secret sec; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((count =3D virSecretEventStateRegisterClient(conn, priv->eventStat= e, secret, eventID, callba= ck, opaque, freecb, &callbackID)) < 0) - goto done; + return -1; =20 /* If this is the first callback for this eventID, we need to enable * events on the server */ @@ -3281,18 +3106,14 @@ remoteConnectSecretEventRegisterAny(virConnectPtr c= onn, (xdrproc_t) xdr_remote_connect_secret_event_register_any_= ret, (char *) &ret) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } =20 virObjectEventStateSetRemote(conn, priv->eventState, callbackID, ret.callbackID); } =20 - rv =3D callbackID; - - done: - remoteDriverUnlock(priv); - return rv; + return callbackID; } =20 =20 @@ -3301,21 +3122,19 @@ remoteConnectSecretEventDeregisterAny(virConnectPtr= conn, int callbackID) { struct private_data *priv =3D conn->privateData; - int rv =3D -1; remote_connect_secret_event_deregister_any_args args; int eventID; int remoteID; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((eventID =3D virObjectEventStateEventID(conn, priv->eventState, callbackID, &remoteID)) < 0) - goto done; + return -1; =20 if ((count =3D virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, true)) < 0) - goto done; + return -1; =20 /* If that was the last callback for this eventID, we need to disable * events on the server */ @@ -3325,15 +3144,11 @@ remoteConnectSecretEventDeregisterAny(virConnectPtr= conn, if (call(conn, priv, 0, REMOTE_PROC_CONNECT_SECRET_EVENT_DEREGISTE= R_ANY, (xdrproc_t) xdr_remote_connect_secret_event_deregister_an= y_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; =20 } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -3346,22 +3161,20 @@ remoteConnectDomainQemuMonitorEventRegister(virConn= ectPtr conn, virFreeCallback freecb, unsigned int flags) { - int rv =3D -1; struct private_data *priv =3D conn->privateData; qemu_connect_domain_monitor_event_register_args args; qemu_connect_domain_monitor_event_register_ret ret; int callbackID; int count; remote_nonnull_domain domain; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((count =3D virDomainQemuMonitorEventStateRegisterID(conn, priv->eventState, dom, event, call= back, opaque, freecb, = -1, &callbackID)) < = 0) - goto done; + return -1; =20 /* If this is the first callback for this event, we need to enable * events on the server */ @@ -3381,17 +3194,13 @@ remoteConnectDomainQemuMonitorEventRegister(virConn= ectPtr conn, (xdrproc_t) xdr_qemu_connect_domain_monitor_event_registe= r_ret, (char *) &ret) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } virObjectEventStateSetRemote(conn, priv->eventState, callbackID, ret.callbackID); } =20 - rv =3D callbackID; - - done: - remoteDriverUnlock(priv); - return rv; + return callbackID; } =20 =20 @@ -3400,20 +3209,18 @@ remoteConnectDomainQemuMonitorEventDeregister(virCo= nnectPtr conn, int callbackID) { struct private_data *priv =3D conn->privateData; - int rv =3D -1; qemu_connect_domain_monitor_event_deregister_args args; int remoteID; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (virObjectEventStateEventID(conn, priv->eventState, callbackID, &remoteID) < 0) - goto done; + return -1; =20 if ((count =3D virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, true)) < 0) - goto done; + return -1; =20 /* If that was the last callback for this event, we need to disable * events on the server */ @@ -3423,14 +3230,10 @@ remoteConnectDomainQemuMonitorEventDeregister(virCo= nnectPtr conn, if (call(conn, priv, REMOTE_CALL_QEMU, QEMU_PROC_CONNECT_DOMAIN_MO= NITOR_EVENT_DEREGISTER, (xdrproc_t) xdr_qemu_connect_domain_monitor_event_deregis= ter_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 /*----------------------------------------------------------------------*/ @@ -3445,8 +3248,7 @@ remoteConnectFindStoragePoolSources(virConnectPtr con= n, remote_connect_find_storage_pool_sources_args args; remote_connect_find_storage_pool_sources_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.type =3D (char*)type; args.srcSpec =3D srcSpec ? (char **)&srcSpec : NULL; @@ -3456,14 +3258,11 @@ remoteConnectFindStoragePoolSources(virConnectPtr c= onn, if (call(conn, priv, 0, REMOTE_PROC_CONNECT_FIND_STORAGE_POOL_SOURCES, (xdrproc_t) xdr_remote_connect_find_storage_pool_sources_args= , (char *) &args, (xdrproc_t) xdr_remote_connect_find_storage_pool_sources_ret,= (char *) &ret) =3D=3D -1) - goto done; + return NULL; =20 rv =3D g_steal_pointer(&ret.xml); /* To stop xdr_free free'ing it */ =20 xdr_free((xdrproc_t) xdr_remote_connect_find_storage_pool_sources_ret,= (char *) &ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -3472,24 +3271,18 @@ remoteConnectFindStoragePoolSources(virConnectPtr c= onn, static int remoteNodeDeviceDettach(virNodeDevicePtr dev) { - int rv =3D -1; remote_node_device_dettach_args args; struct private_data *priv =3D dev->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.name =3D dev->name; =20 if (call(dev->conn, priv, 0, REMOTE_PROC_NODE_DEVICE_DETTACH, (xdrproc_t) xdr_remote_node_device_dettach_args, (char *) &ar= gs, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; - - rv =3D 0; + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static int @@ -3497,11 +3290,9 @@ remoteNodeDeviceDetachFlags(virNodeDevicePtr dev, const char *driverName, unsigned int flags) { - int rv =3D -1; remote_node_device_detach_flags_args args; struct private_data *priv =3D dev->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.name =3D dev->name; args.driverName =3D driverName ? (char**)&driverName : NULL; @@ -3510,61 +3301,45 @@ remoteNodeDeviceDetachFlags(virNodeDevicePtr dev, if (call(dev->conn, priv, 0, REMOTE_PROC_NODE_DEVICE_DETACH_FLAGS, (xdrproc_t) xdr_remote_node_device_detach_flags_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -= 1) - goto done; - - rv =3D 0; + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static int remoteNodeDeviceReAttach(virNodeDevicePtr dev) { - int rv =3D -1; remote_node_device_re_attach_args args; struct private_data *priv =3D dev->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.name =3D dev->name; =20 if (call(dev->conn, priv, 0, REMOTE_PROC_NODE_DEVICE_RE_ATTACH, (xdrproc_t) xdr_remote_node_device_re_attach_args, (char *) &= args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; - - rv =3D 0; + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static int remoteNodeDeviceReset(virNodeDevicePtr dev) { - int rv =3D -1; remote_node_device_reset_args args; /* This method is unusual in that it uses the HV driver, not the devMo= n driver * hence its use of privateData, instead of nodeDevicePrivateData */ struct private_data *priv =3D dev->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.name =3D dev->name; =20 if (call(dev->conn, priv, 0, REMOTE_PROC_NODE_DEVICE_RESET, (xdrproc_t) xdr_remote_node_device_reset_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; - - rv =3D 0; + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -4229,11 +4004,9 @@ remoteConnectDomainEventRegister(virConnectPtr conn, virFreeCallback freecb) { int callbackID; - int rv =3D -1; struct private_data *priv =3D conn->privateData; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((count =3D virDomainEventStateRegisterClient(conn, priv->eventStat= e, NULL, @@ -4242,7 +4015,7 @@ remoteConnectDomainEventRegister(virConnectPtr conn, opaque, freecb, true, &callbackID, priv->serverEventFilter= )) < 0) - goto done; + return -1; =20 if (count =3D=3D 1) { /* Tell the server when we are the first callback registering */ @@ -4260,7 +4033,7 @@ remoteConnectDomainEventRegister(virConnectPtr conn, (xdrproc_t) xdr_remote_connect_domain_event_callback_= register_any_ret, (char *) &ret) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } virObjectEventStateSetRemote(conn, priv->eventState, callbackI= D, ret.callbackID); @@ -4270,16 +4043,12 @@ remoteConnectDomainEventRegister(virConnectPtr conn, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } } } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -4288,22 +4057,20 @@ remoteConnectDomainEventDeregister(virConnectPtr co= nn, virConnectDomainEventCallback callback) { struct private_data *priv =3D conn->privateData; - int rv =3D -1; remote_connect_domain_event_callback_deregister_any_args args; int callbackID; int remoteID; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((callbackID =3D virDomainEventStateCallbackID(conn, priv->eventSta= te, callback, &remoteID)) < 0) - goto done; + return -1; =20 if ((count =3D virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, true)) < 0) - goto done; + return -1; =20 if (count =3D=3D 0) { /* Tell the server when we are the last callback deregistering */ @@ -4314,20 +4081,16 @@ remoteConnectDomainEventDeregister(virConnectPtr co= nn, REMOTE_PROC_CONNECT_DOMAIN_EVENT_CALLBACK_DEREGISTER_= ANY, (xdrproc_t) xdr_remote_connect_domain_event_callback_= deregister_any_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; } else { if (call(conn, priv, 0, REMOTE_PROC_CONNECT_DOMAIN_EVENT_DEREG= ISTER, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; } } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -5367,13 +5130,10 @@ static unsigned char * remoteSecretGetValue(virSecretPtr secret, size_t *value_size, unsigned int flags) { - unsigned char *rv =3D NULL; remote_secret_get_value_args args; remote_secret_get_value_ret ret; struct private_data *priv =3D secret->conn->privateData; =20 - remoteDriverLock(priv); - make_nonnull_secret(&args.secret, secret); args.flags =3D flags; =20 @@ -5381,14 +5141,10 @@ remoteSecretGetValue(virSecretPtr secret, size_t *v= alue_size, if (call(secret->conn, priv, 0, REMOTE_PROC_SECRET_GET_VALUE, (xdrproc_t) xdr_remote_secret_get_value_args, (char *) &args, (xdrproc_t) xdr_remote_secret_get_value_ret, (char *) &ret) = =3D=3D -1) - goto done; + return NULL; =20 *value_size =3D ret.value.value_len; - rv =3D (unsigned char *) ret.value.value_val; /* Caller frees. */ - - done: - remoteDriverUnlock(priv); - return rv; + return (unsigned char *) ret.value.value_val; /* Caller frees. */ } =20 =20 @@ -5654,6 +5410,7 @@ remoteStreamEventAddCallback(virStreamPtr st, virNetClientStream *privst =3D st->privateData; int ret =3D -1; struct remoteStreamCallbackData *cbdata; + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 cbdata =3D g_new0(struct remoteStreamCallbackData, 1); cbdata->cb =3D cb; @@ -5662,19 +5419,15 @@ remoteStreamEventAddCallback(virStreamPtr st, cbdata->st =3D st; virStreamRef(st); =20 - remoteDriverLock(priv); - if ((ret =3D virNetClientStreamEventAddCallback(privst, events, remoteStreamEventCallbac= k, cbdata, remoteStreamCallbackFree= )) < 0) { VIR_FREE(cbdata); - goto cleanup; + return -1; } =20 - cleanup: - remoteDriverUnlock(priv); return ret; } =20 @@ -5685,14 +5438,9 @@ remoteStreamEventUpdateCallback(virStreamPtr st, { struct private_data *priv =3D st->conn->privateData; virNetClientStream *privst =3D st->privateData; - int ret =3D -1; - - remoteDriverLock(priv); - - ret =3D virNetClientStreamEventUpdateCallback(privst, events); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - remoteDriverUnlock(priv); - return ret; + return virNetClientStreamEventUpdateCallback(privst, events); } =20 =20 @@ -5701,14 +5449,9 @@ remoteStreamEventRemoveCallback(virStreamPtr st) { struct private_data *priv =3D st->conn->privateData; virNetClientStream *privst =3D st->privateData; - int ret =3D -1; - - remoteDriverLock(priv); - - ret =3D virNetClientStreamEventRemoveCallback(privst); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - remoteDriverUnlock(priv); - return ret; + return virNetClientStreamEventRemoveCallback(privst); } =20 =20 @@ -5777,20 +5520,18 @@ remoteConnectDomainEventRegisterAny(virConnectPtr c= onn, void *opaque, virFreeCallback freecb) { - int rv =3D -1; struct private_data *priv =3D conn->privateData; int callbackID; int count; remote_nonnull_domain domain; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((count =3D virDomainEventStateRegisterClient(conn, priv->eventStat= e, dom, eventID, callback, opaque, freecb, false, &callbackID, priv->serverEventFilter= )) < 0) - goto done; + return -1; =20 /* If this is the first callback for this eventID, we need to enable * events on the server */ @@ -5813,7 +5554,7 @@ remoteConnectDomainEventRegisterAny(virConnectPtr con= n, (xdrproc_t) xdr_remote_connect_domain_event_callback_= register_any_ret, (char *) &ret) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } virObjectEventStateSetRemote(conn, priv->eventState, callbackI= D, ret.callbackID); @@ -5827,16 +5568,12 @@ remoteConnectDomainEventRegisterAny(virConnectPtr c= onn, (xdrproc_t) xdr_void, (char *)NULL) =3D=3D -1) { virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, false); - goto done; + return -1; } } } =20 - rv =3D callbackID; - - done: - remoteDriverUnlock(priv); - return rv; + return callbackID; } =20 =20 @@ -5845,20 +5582,18 @@ remoteConnectDomainEventDeregisterAny(virConnectPtr= conn, int callbackID) { struct private_data *priv =3D conn->privateData; - int rv =3D -1; int eventID; int remoteID; int count; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if ((eventID =3D virObjectEventStateEventID(conn, priv->eventState, callbackID, &remoteID)) < 0) - goto done; + return -1; =20 if ((count =3D virObjectEventStateDeregisterID(conn, priv->eventState, callbackID, true)) < 0) - goto done; + return -1; =20 /* If that was the last callback for this eventID, we need to disable * events on the server */ @@ -5872,7 +5607,7 @@ remoteConnectDomainEventDeregisterAny(virConnectPtr c= onn, REMOTE_PROC_CONNECT_DOMAIN_EVENT_CALLBACK_DEREGISTER_= ANY, (xdrproc_t) xdr_remote_connect_domain_event_callback_= deregister_any_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; } else { remote_connect_domain_event_deregister_any_args args; =20 @@ -5881,15 +5616,11 @@ remoteConnectDomainEventDeregisterAny(virConnectPtr= conn, if (call(conn, priv, 0, REMOTE_PROC_CONNECT_DOMAIN_EVENT_DEREG= ISTER_ANY, (xdrproc_t) xdr_remote_connect_domain_event_deregiste= r_any_args, (char *) &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; + return -1; } } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -5903,8 +5634,7 @@ remoteDomainQemuMonitorCommand(virDomainPtr domain, c= onst char *cmd, qemu_domain_monitor_command_args args; qemu_domain_monitor_command_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.cmd =3D (char *)cmd; @@ -5914,7 +5644,7 @@ remoteDomainQemuMonitorCommand(virDomainPtr domain, c= onst char *cmd, if (call(domain->conn, priv, REMOTE_CALL_QEMU, QEMU_PROC_DOMAIN_MONITO= R_COMMAND, (xdrproc_t) xdr_qemu_domain_monitor_command_args, (char *) &a= rgs, (xdrproc_t) xdr_qemu_domain_monitor_command_ret, (char *) &re= t) =3D=3D -1) - goto done; + return -1; =20 *result =3D g_strdup(ret.result); =20 @@ -5922,8 +5652,6 @@ remoteDomainQemuMonitorCommand(virDomainPtr domain, c= onst char *cmd, =20 xdr_free((xdrproc_t) xdr_qemu_domain_monitor_command_ret, (char *) &re= t); =20 - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -5938,15 +5666,13 @@ remoteDomainQemuMonitorCommandWithFiles(virDomainPt= r domain, char **result, unsigned int flags) { - int rv =3D -1; qemu_domain_monitor_command_with_files_args args; qemu_domain_monitor_command_with_files_ret ret; struct private_data *priv =3D domain->conn->privateData; size_t rpc_noutfiles =3D 0; g_autofree int *rpc_outfiles =3D NULL; size_t i; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.cmd =3D (char *)cmd; @@ -5958,7 +5684,7 @@ remoteDomainQemuMonitorCommandWithFiles(virDomainPtr = domain, QEMU_PROC_DOMAIN_MONITOR_COMMAND_WITH_FILES, (xdrproc_t) xdr_qemu_domain_monitor_command_with_files_ar= gs, (char *) &args, (xdrproc_t) xdr_qemu_domain_monitor_command_with_files_re= t, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (outfiles) *outfiles =3D g_steal_pointer(&rpc_outfiles); @@ -5968,19 +5694,15 @@ remoteDomainQemuMonitorCommandWithFiles(virDomainPt= r domain, =20 *result =3D g_strdup(ret.result); =20 - rv =3D 0; - xdr_free((xdrproc_t) xdr_qemu_domain_monitor_command_with_files_ret, (= char *) &ret); =20 - done: if (rpc_outfiles) { for (i =3D 0; i < rpc_noutfiles; i++) { VIR_FORCE_CLOSE(rpc_outfiles[i]); } } =20 - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -5997,8 +5719,7 @@ remoteDomainMigrateBegin3(virDomainPtr domain, remote_domain_migrate_begin3_args args; remote_domain_migrate_begin3_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -6011,7 +5732,7 @@ remoteDomainMigrateBegin3(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_BEGIN3, (xdrproc_t) xdr_remote_domain_migrate_begin3_args, (char *) &= args, (xdrproc_t) xdr_remote_domain_migrate_begin3_ret, (char *) &r= et) =3D=3D -1) - goto done; + return NULL; =20 if (ret.cookie_out.cookie_out_len > 0) { if (!cookieout || !cookieoutlen) { @@ -6026,7 +5747,6 @@ remoteDomainMigrateBegin3(virDomainPtr domain, rv =3D ret.xml; /* caller frees */ =20 done: - remoteDriverUnlock(priv); return rv; =20 error: @@ -6052,8 +5772,7 @@ remoteDomainMigratePrepare3(virConnectPtr dconn, remote_domain_migrate_prepare3_args args; remote_domain_migrate_prepare3_ret ret; struct private_data *priv =3D dconn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -6069,7 +5788,7 @@ remoteDomainMigratePrepare3(virConnectPtr dconn, if (call(dconn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE3, (xdrproc_t) xdr_remote_domain_migrate_prepare3_args, (char *)= &args, (xdrproc_t) xdr_remote_domain_migrate_prepare3_ret, (char *) = &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.cookie_out.cookie_out_len > 0) { if (!cookieout || !cookieoutlen) { @@ -6093,7 +5812,6 @@ remoteDomainMigratePrepare3(virConnectPtr dconn, =20 done: VIR_FREE(ret.uri_out); - remoteDriverUnlock(priv); return rv; error: VIR_FREE(ret.cookie_out.cookie_out_val); @@ -6116,12 +5834,10 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dco= nn, const char *dom_xml) { struct private_data *priv =3D dconn->privateData; - int rv =3D -1; remote_domain_migrate_prepare_tunnel3_args args; remote_domain_migrate_prepare_tunnel3_ret ret; virNetClientStream *netst; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -6129,11 +5845,11 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dco= nn, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE= _TUNNEL3, priv->counter, false))) - goto done; + return -1; =20 if (virNetClientAddStream(priv->client, netst) < 0) { virObjectUnref(netst); - goto done; + return -1; } =20 st->driver =3D &remoteStreamDrv; @@ -6152,7 +5868,7 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dconn, (xdrproc_t) xdr_remote_domain_migrate_prepare_tunnel3_ret, (c= har *) &ret) =3D=3D -1) { virNetClientRemoveStream(priv->client, netst); virObjectUnref(netst); - goto done; + return -1; } =20 if (ret.cookie_out.cookie_out_len > 0) { @@ -6165,15 +5881,11 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dco= nn, *cookieoutlen =3D ret.cookie_out.cookie_out_len; } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; =20 error: VIR_FREE(ret.cookie_out.cookie_out_val); - goto done; + return -1; } =20 =20 @@ -6190,12 +5902,10 @@ remoteDomainMigratePerform3(virDomainPtr dom, const char *dname, unsigned long resource) { - int rv =3D -1; remote_domain_migrate_perform3_args args; remote_domain_migrate_perform3_ret ret; struct private_data *priv =3D dom->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -6213,7 +5923,7 @@ remoteDomainMigratePerform3(virDomainPtr dom, if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_PERFORM3, (xdrproc_t) xdr_remote_domain_migrate_perform3_args, (char *)= &args, (xdrproc_t) xdr_remote_domain_migrate_perform3_ret, (char *) = &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.cookie_out.cookie_out_len > 0) { if (!cookieout || !cookieoutlen) { @@ -6225,15 +5935,11 @@ remoteDomainMigratePerform3(virDomainPtr dom, *cookieoutlen =3D ret.cookie_out.cookie_out_len; } =20 - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; =20 error: VIR_FREE(ret.cookie_out.cookie_out_val); - goto done; + return -1; } =20 =20 @@ -6253,8 +5959,7 @@ remoteDomainMigrateFinish3(virConnectPtr dconn, remote_domain_migrate_finish3_ret ret; struct private_data *priv =3D dconn->privateData; virDomainPtr rv =3D NULL; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -6269,7 +5974,7 @@ remoteDomainMigrateFinish3(virConnectPtr dconn, if (call(dconn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_FINISH3, (xdrproc_t) xdr_remote_domain_migrate_finish3_args, (char *) = &args, (xdrproc_t) xdr_remote_domain_migrate_finish3_ret, (char *) &= ret) =3D=3D -1) - goto done; + return NULL; =20 rv =3D get_nonnull_domain(dconn, ret.dom); =20 @@ -6286,13 +5991,11 @@ remoteDomainMigrateFinish3(virConnectPtr dconn, =20 xdr_free((xdrproc_t) &xdr_remote_domain_migrate_finish3_ret, (char *) = &ret); =20 - done: - remoteDriverUnlock(priv); return rv; =20 error: VIR_FREE(ret.cookie_out.cookie_out_val); - goto done; + return NULL; } =20 =20 @@ -6303,11 +6006,9 @@ remoteDomainMigrateConfirm3(virDomainPtr domain, unsigned long flags, int cancelled) { - int rv =3D -1; remote_domain_migrate_confirm3_args args; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.cookie_in.cookie_in_len =3D cookieinlen; @@ -6318,13 +6019,9 @@ remoteDomainMigrateConfirm3(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_CONFIRM3, (xdrproc_t) xdr_remote_domain_migrate_confirm3_args, (char *)= &args, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto done; - - rv =3D 0; + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -6339,10 +6036,8 @@ remoteConnectGetCPUModelNames(virConnectPtr conn, g_auto(GStrv) retmodels =3D NULL; remote_connect_get_cpu_model_names_args args; remote_connect_get_cpu_model_names_ret ret; - struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.arch =3D (char *) arch; args.need_results =3D !!models; @@ -6354,7 +6049,7 @@ remoteConnectGetCPUModelNames(virConnectPtr conn, (char *) &args, (xdrproc_t) xdr_remote_connect_get_cpu_model_names_ret, (char *) &ret) < 0) - goto done; + return -1; =20 /* Check the length of the returned list carefully. */ if (ret.models.models_len > REMOTE_CONNECT_CPU_MODELS_MAX) { @@ -6379,8 +6074,6 @@ remoteConnectGetCPUModelNames(virConnectPtr conn, cleanup: xdr_free((xdrproc_t) xdr_remote_connect_get_cpu_model_names_ret, (char= *) &ret); =20 - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -6391,13 +6084,11 @@ remoteDomainOpenGraphics(virDomainPtr dom, int fd, unsigned int flags) { - int rv =3D -1; remote_domain_open_graphics_args args; struct private_data *priv =3D dom->conn->privateData; int fdin[] =3D { fd }; size_t fdinlen =3D G_N_ELEMENTS(fdin); - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); args.idx =3D idx; @@ -6409,14 +6100,9 @@ remoteDomainOpenGraphics(virDomainPtr dom, REMOTE_PROC_DOMAIN_OPEN_GRAPHICS, (xdrproc_t) xdr_remote_domain_open_graphics_args, (char *= ) &args, (xdrproc_t) xdr_void, NULL) =3D=3D -1) - goto done; - - rv =3D 0; - - done: - remoteDriverUnlock(priv); + return -1; =20 - return rv; + return 0; } =20 =20 @@ -6430,8 +6116,7 @@ remoteDomainOpenGraphicsFD(virDomainPtr dom, struct private_data *priv =3D dom->conn->privateData; int *fdout =3D NULL; size_t fdoutlen =3D 0; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); args.idx =3D idx; @@ -6443,7 +6128,7 @@ remoteDomainOpenGraphicsFD(virDomainPtr dom, REMOTE_PROC_DOMAIN_OPEN_GRAPHICS_FD, (xdrproc_t) xdr_remote_domain_open_graphics_fd_args, (cha= r *) &args, (xdrproc_t) xdr_void, NULL) =3D=3D -1) - goto done; + return -1; =20 if (fdoutlen !=3D 1) { if (fdoutlen) { @@ -6455,14 +6140,11 @@ remoteDomainOpenGraphicsFD(virDomainPtr dom, virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("no file descriptor received")); } - goto done; + return -1; } rv =3D fdout[0]; =20 - done: VIR_FREE(fdout); - remoteDriverUnlock(priv); - return rv; } =20 @@ -6471,31 +6153,25 @@ static int remoteConnectSetKeepAlive(virConnectPtr conn, int interval, unsigned int c= ount) { struct private_data *priv =3D conn->privateData; - int ret =3D -1; + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - remoteDriverLock(priv); if (!virNetClientKeepAliveIsSupported(priv->client)) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("the caller doesn't support keepalive protocol;" " perhaps it's missing event loop implementation"= )); - goto cleanup; + return -1; } =20 if (!priv->serverKeepAlive) { - ret =3D 1; - goto cleanup; + return 1; } =20 if (interval > 0) { - ret =3D virNetClientKeepAliveStart(priv->client, interval, count); + return virNetClientKeepAliveStart(priv->client, interval, count); } else { virNetClientKeepAliveStop(priv->client); - ret =3D 0; + return 0; } - - cleanup: - remoteDriverUnlock(priv); - return ret; } =20 =20 @@ -6504,10 +6180,9 @@ remoteConnectIsAlive(virConnectPtr conn) { struct private_data *priv =3D conn->privateData; bool ret; + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - remoteDriverLock(priv); ret =3D virNetClientIsOpen(priv->client); - remoteDriverUnlock(priv); =20 if (ret) return 1; @@ -6526,8 +6201,7 @@ remoteDomainGetDiskErrors(virDomainPtr dom, struct private_data *priv =3D dom->conn->privateData; remote_domain_get_disk_errors_args args; remote_domain_get_disk_errors_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); args.maxerrors =3D maxerrors; @@ -6540,7 +6214,7 @@ remoteDomainGetDiskErrors(virDomainPtr dom, (char *) &args, (xdrproc_t) xdr_remote_domain_get_disk_errors_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (remoteDeserializeDomainDiskErrors(ret.errors.errors_val, ret.errors.errors_len, @@ -6553,9 +6227,6 @@ remoteDomainGetDiskErrors(virDomainPtr dom, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_disk_errors_ret, (char *) &= ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -6638,8 +6309,7 @@ remoteDomainGetInterfaceParameters(virDomainPtr domai= n, remote_domain_get_interface_parameters_args args; remote_domain_get_interface_parameters_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.device =3D (char *)device; @@ -6650,7 +6320,7 @@ remoteDomainGetInterfaceParameters(virDomainPtr domai= n, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_INTERFACE_PARAM= ETERS, (xdrproc_t) xdr_remote_domain_get_interface_parameters_args, = (char *) &args, (xdrproc_t) xdr_remote_domain_get_interface_parameters_ret, (= char *) &ret) =3D=3D -1) - goto done; + return -1; =20 /* Handle the case when the caller does not know the number of paramet= ers * and is asking for the number of parameters supported @@ -6673,8 +6343,6 @@ remoteDomainGetInterfaceParameters(virDomainPtr domai= n, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_interface_parameters_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -6689,8 +6357,7 @@ remoteNodeGetMemoryParameters(virConnectPtr conn, remote_node_get_memory_parameters_args args; remote_node_get_memory_parameters_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.nparams =3D *nparams; args.flags =3D flags; @@ -6699,7 +6366,7 @@ remoteNodeGetMemoryParameters(virConnectPtr conn, if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_MEMORY_PARAMETERS, (xdrproc_t) xdr_remote_node_get_memory_parameters_args, (char= *) &args, (xdrproc_t) xdr_remote_node_get_memory_parameters_ret, (char = *) &ret) =3D=3D -1) - goto done; + return -1; =20 /* Handle the case when the caller does not know the number of paramet= ers * and is asking for the number of parameters supported @@ -6722,8 +6389,6 @@ remoteNodeGetMemoryParameters(virConnectPtr conn, cleanup: xdr_free((xdrproc_t) xdr_remote_node_get_memory_parameters_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -6738,8 +6403,7 @@ remoteNodeGetSEVInfo(virConnectPtr conn, remote_node_get_sev_info_args args; remote_node_get_sev_info_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.flags =3D flags; =20 @@ -6747,7 +6411,7 @@ remoteNodeGetSEVInfo(virConnectPtr conn, if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_SEV_INFO, (xdrproc_t) xdr_remote_node_get_sev_info_args, (char *) &args, (xdrproc_t) xdr_remote_node_get_sev_info_ret, (char *) &ret) = =3D=3D -1) - goto done; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -6760,8 +6424,6 @@ remoteNodeGetSEVInfo(virConnectPtr conn, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_node_get_sev_info_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -6776,8 +6438,7 @@ remoteNodeGetCPUMap(virConnectPtr conn, remote_node_get_cpu_map_args args; remote_node_get_cpu_map_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.need_map =3D !!cpumap; args.need_online =3D !!online; @@ -6789,7 +6450,7 @@ remoteNodeGetCPUMap(virConnectPtr conn, (char *) &args, (xdrproc_t) xdr_remote_node_get_cpu_map_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (ret.ret < 0) goto cleanup; @@ -6806,8 +6467,6 @@ remoteNodeGetCPUMap(virConnectPtr conn, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_node_get_cpu_map_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -6817,12 +6476,10 @@ remoteDomainLxcOpenNamespace(virDomainPtr domain, int **fdlist, unsigned int flags) { - int rv =3D -1; lxc_domain_open_namespace_args args; struct private_data *priv =3D domain->conn->privateData; size_t nfds =3D 0; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; @@ -6835,13 +6492,9 @@ remoteDomainLxcOpenNamespace(virDomainPtr domain, LXC_PROC_DOMAIN_OPEN_NAMESPACE, (xdrproc_t) xdr_lxc_domain_open_namespace_args, (char *) = &args, (xdrproc_t) xdr_void, NULL) =3D=3D -1) - goto done; - - rv =3D nfds; + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return nfds; } =20 static int @@ -6855,8 +6508,7 @@ remoteDomainGetJobStats(virDomainPtr domain, remote_domain_get_job_stats_args args; remote_domain_get_job_stats_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; @@ -6865,7 +6517,7 @@ remoteDomainGetJobStats(virDomainPtr domain, if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_JOB_STATS, (xdrproc_t) xdr_remote_domain_get_job_stats_args, (char *) &a= rgs, (xdrproc_t) xdr_remote_domain_get_job_stats_ret, (char *) &re= t) =3D=3D -1) - goto done; + return -1; =20 *type =3D ret.type; =20 @@ -6880,8 +6532,6 @@ remoteDomainGetJobStats(virDomainPtr domain, cleanup: xdr_free((xdrproc_t) xdr_remote_domain_get_job_stats_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -6898,8 +6548,7 @@ remoteDomainMigrateBegin3Params(virDomainPtr domain, remote_domain_migrate_begin3_params_args args; remote_domain_migrate_begin3_params_ret ret; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -6936,7 +6585,6 @@ remoteDomainMigrateBegin3Params(virDomainPtr domain, cleanup: virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); - remoteDriverUnlock(priv); return rv; =20 error: @@ -6960,8 +6608,7 @@ remoteDomainMigratePrepare3Params(virConnectPtr dconn, remote_domain_migrate_prepare3_params_args args; remote_domain_migrate_prepare3_params_ret ret; struct private_data *priv =3D dconn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -7008,7 +6655,6 @@ remoteDomainMigratePrepare3Params(virConnectPtr dconn, virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); VIR_FREE(ret.uri_out); - remoteDriverUnlock(priv); return rv; =20 error: @@ -7035,8 +6681,7 @@ remoteDomainMigratePrepareTunnel3Params(virConnectPtr= dconn, remote_domain_migrate_prepare_tunnel3_params_args args; remote_domain_migrate_prepare_tunnel3_params_ret ret; virNetClientStream *netst; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -7092,7 +6737,6 @@ remoteDomainMigratePrepareTunnel3Params(virConnectPtr= dconn, cleanup: virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); - remoteDriverUnlock(priv); return rv; =20 error: @@ -7116,8 +6760,7 @@ remoteDomainMigratePerform3Params(virDomainPtr dom, remote_domain_migrate_perform3_params_args args; remote_domain_migrate_perform3_params_ret ret; struct private_data *priv =3D dom->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -7157,7 +6800,6 @@ remoteDomainMigratePerform3Params(virDomainPtr dom, cleanup: virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); - remoteDriverUnlock(priv); return rv; =20 error: @@ -7181,8 +6823,7 @@ remoteDomainMigrateFinish3Params(virConnectPtr dconn, remote_domain_migrate_finish3_params_ret ret; struct private_data *priv =3D dconn->privateData; virDomainPtr rv =3D NULL; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 memset(&ret, 0, sizeof(ret)); =20 @@ -7225,7 +6866,6 @@ remoteDomainMigrateFinish3Params(virConnectPtr dconn, cleanup: virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); - remoteDriverUnlock(priv); return rv; =20 error: @@ -7246,8 +6886,7 @@ remoteDomainMigrateConfirm3Params(virDomainPtr domain, int rv =3D -1; remote_domain_migrate_confirm3_params_args args; struct private_data *priv =3D domain->conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.cookie_in.cookie_in_len =3D cookieinlen; @@ -7273,7 +6912,6 @@ remoteDomainMigrateConfirm3Params(virDomainPtr domain, cleanup: virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); - remoteDriverUnlock(priv); return rv; } =20 @@ -7286,8 +6924,7 @@ remoteDomainCreateXMLWithFiles(virConnectPtr conn, co= nst char *xml_desc, struct private_data *priv =3D conn->privateData; remote_domain_create_xml_with_files_args args; remote_domain_create_xml_with_files_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.xml_desc =3D (char *)xml_desc; args.flags =3D flags; @@ -7299,15 +6936,12 @@ remoteDomainCreateXMLWithFiles(virConnectPtr conn, = const char *xml_desc, NULL, NULL, REMOTE_PROC_DOMAIN_CREATE_XML_WITH_FILES, (xdrproc_t)xdr_remote_domain_create_xml_with_files_args, = (char *)&args, - (xdrproc_t)xdr_remote_domain_create_xml_with_files_ret, (= char *)&ret) =3D=3D -1) { - goto done; - } + (xdrproc_t)xdr_remote_domain_create_xml_with_files_ret, (= char *)&ret) =3D=3D -1) + return NULL; =20 rv =3D get_nonnull_domain(conn, ret.dom); xdr_free((xdrproc_t)xdr_remote_domain_create_xml_with_files_ret, (char= *)&ret); =20 - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -7317,12 +6951,10 @@ remoteDomainCreateWithFiles(virDomainPtr dom, unsigned int nfiles, int *files, unsigned int flags) { - int rv =3D -1; struct private_data *priv =3D dom->conn->privateData; remote_domain_create_with_files_args args; remote_domain_create_with_files_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); args.flags =3D flags; @@ -7334,17 +6966,12 @@ remoteDomainCreateWithFiles(virDomainPtr dom, NULL, NULL, REMOTE_PROC_DOMAIN_CREATE_WITH_FILES, (xdrproc_t)xdr_remote_domain_create_with_files_args, (cha= r *)&args, - (xdrproc_t)xdr_remote_domain_create_with_files_ret, (char= *)&ret) =3D=3D -1) { - goto done; - } + (xdrproc_t)xdr_remote_domain_create_with_files_ret, (char= *)&ret) =3D=3D -1) + return -1; =20 dom->id =3D ret.dom.id; xdr_free((xdrproc_t) &xdr_remote_domain_create_with_files_ret, (char *= ) &ret); - rv =3D 0; - - done: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 static int @@ -7353,12 +6980,10 @@ remoteDomainGetTime(virDomainPtr dom, unsigned int *nseconds, unsigned int flags) { - int rv =3D -1; struct private_data *priv =3D dom->conn->privateData; remote_domain_get_time_args args; remote_domain_get_time_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); args.flags =3D flags; @@ -7368,16 +6993,12 @@ remoteDomainGetTime(virDomainPtr dom, if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_TIME, (xdrproc_t) xdr_remote_domain_get_time_args, (char *) &args, (xdrproc_t) xdr_remote_domain_get_time_ret, (char *) &ret) = =3D=3D -1) - goto cleanup; + return -1; =20 *seconds =3D ret.seconds; *nseconds =3D ret.nseconds; xdr_free((xdrproc_t) &xdr_remote_domain_get_time_ret, (char *) &ret); - rv =3D 0; - - cleanup: - remoteDriverUnlock(priv); - return rv; + return 0; } =20 =20 @@ -7390,18 +7011,16 @@ remoteNodeGetFreePages(virConnectPtr conn, unsigned long long *counts, unsigned int flags) { - int rv =3D -1; remote_node_get_free_pages_args args; remote_node_get_free_pages_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (npages * cellCount > REMOTE_NODE_MAX_CELLS) { virReportError(VIR_ERR_RPC, _("too many NUMA cells: %d > %d"), npages * cellCount, REMOTE_NODE_MAX_CELLS); - goto done; + return -1; } =20 args.pages.pages_val =3D (u_int *) pages; @@ -7414,17 +7033,13 @@ remoteNodeGetFreePages(virConnectPtr conn, if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_FREE_PAGES, (xdrproc_t) xdr_remote_node_get_free_pages_args, (char *)&arg= s, (xdrproc_t) xdr_remote_node_get_free_pages_ret, (char *)&ret)= =3D=3D -1) - goto done; + return -1; =20 memcpy(counts, ret.counts.counts_val, ret.counts.counts_len * sizeof(*= counts)); =20 xdr_free((xdrproc_t) xdr_remote_node_get_free_pages_ret, (char *) &ret= ); =20 - rv =3D ret.counts.counts_len; - - done: - remoteDriverUnlock(priv); - return rv; + return ret.counts.counts_len; } =20 =20 @@ -7479,9 +7094,8 @@ remoteNetworkGetDHCPLeases(virNetworkPtr net, struct private_data *priv =3D net->conn->privateData; remote_network_get_dhcp_leases_args args; remote_network_get_dhcp_leases_ret ret; - virNetworkDHCPLeasePtr *leases_ret =3D NULL; - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_network(&args.net, net); args.mac =3D mac ? (char **) &mac : NULL; @@ -7493,7 +7107,7 @@ remoteNetworkGetDHCPLeases(virNetworkPtr net, if (call(net->conn, priv, 0, REMOTE_PROC_NETWORK_GET_DHCP_LEASES, (xdrproc_t)xdr_remote_network_get_dhcp_leases_args, (char *)&= args, (xdrproc_t)xdr_remote_network_get_dhcp_leases_ret, (char *)&r= et) =3D=3D -1) - goto done; + return -1; =20 if (ret.leases.leases_len > REMOTE_NETWORK_DHCP_LEASES_MAX) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -7527,8 +7141,6 @@ remoteNetworkGetDHCPLeases(virNetworkPtr net, xdr_free((xdrproc_t)xdr_remote_network_get_dhcp_leases_ret, (char *) &ret); =20 - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -7548,6 +7160,7 @@ remoteConnectGetAllDomainStats(virConnectPtr conn, remote_connect_get_all_domain_stats_ret ret; virDomainStatsRecordPtr elem =3D NULL; virDomainStatsRecordPtr *tmpret =3D NULL; + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (ndoms) { args.doms.doms_val =3D g_new0(remote_nonnull_domain, ndoms); @@ -7562,14 +7175,11 @@ remoteConnectGetAllDomainStats(virConnectPtr conn, =20 memset(&ret, 0, sizeof(ret)); =20 - remoteDriverLock(priv); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_GET_ALL_DOMAIN_STATS, (xdrproc_t)xdr_remote_connect_get_all_domain_stats_args, (cha= r *)&args, (xdrproc_t)xdr_remote_connect_get_all_domain_stats_ret, (char= *)&ret) =3D=3D -1) { - remoteDriverUnlock(priv); goto cleanup; } - remoteDriverUnlock(priv); =20 if (ret.retStats.retStats_len > REMOTE_DOMAIN_LIST_MAX) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -7626,18 +7236,16 @@ remoteNodeAllocPages(virConnectPtr conn, unsigned int cellCount, unsigned int flags) { - int rv =3D -1; remote_node_alloc_pages_args args; remote_node_alloc_pages_ret ret; struct private_data *priv =3D conn->privateData; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (npages > REMOTE_NODE_MAX_CELLS) { virReportError(VIR_ERR_RPC, _("too many NUMA cells: %d > %d"), npages, REMOTE_NODE_MAX_CELLS); - goto done; + return -1; } =20 args.pageSizes.pageSizes_val =3D (u_int *) pageSizes; @@ -7652,13 +7260,9 @@ remoteNodeAllocPages(virConnectPtr conn, if (call(conn, priv, 0, REMOTE_PROC_NODE_ALLOC_PAGES, (xdrproc_t) xdr_remote_node_alloc_pages_args, (char *) &args, (xdrproc_t) xdr_remote_node_alloc_pages_ret, (char *) &ret) = =3D=3D -1) - goto done; - - rv =3D ret.ret; + return -1; =20 - done: - remoteDriverUnlock(priv); - return rv; + return ret.ret; } =20 =20 @@ -7673,8 +7277,7 @@ remoteDomainGetFSInfo(virDomainPtr dom, remote_domain_get_fsinfo_args args; remote_domain_get_fsinfo_ret ret; remote_domain_fsinfo *src; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); =20 @@ -7685,7 +7288,7 @@ remoteDomainGetFSInfo(virDomainPtr dom, if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_FSINFO, (xdrproc_t)xdr_remote_domain_get_fsinfo_args, (char *)&args, (xdrproc_t)xdr_remote_domain_get_fsinfo_ret, (char *)&ret) = =3D=3D -1) - goto done; + return -1; =20 if (ret.info.info_len > REMOTE_DOMAIN_FSINFO_MAX) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -7733,9 +7336,6 @@ remoteDomainGetFSInfo(virDomainPtr dom, cleanup: xdr_free((xdrproc_t)xdr_remote_domain_get_fsinfo_ret, (char *) &ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -7752,24 +7352,21 @@ remoteDomainInterfaceAddresses(virDomainPtr dom, virDomainInterfacePtr *ifaces_ret =3D NULL; remote_domain_interface_addresses_args args; remote_domain_interface_addresses_ret ret; - struct private_data *priv =3D dom->conn->privateData; + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.source =3D source; args.flags =3D flags; make_nonnull_domain(&args.dom, dom); =20 - remoteDriverLock(priv); - memset(&ret, 0, sizeof(ret)); =20 if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_INTERFACE_ADDRESSES, (xdrproc_t)xdr_remote_domain_interface_addresses_args, (char *)&args, (xdrproc_t)xdr_remote_domain_interface_addresses_ret, - (char *)&ret) =3D=3D -1) { - goto done; - } + (char *)&ret) =3D=3D -1) + return -1; =20 if (ret.ifaces.ifaces_len > REMOTE_DOMAIN_INTERFACE_MAX) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -7830,8 +7427,6 @@ remoteDomainInterfaceAddresses(virDomainPtr dom, } xdr_free((xdrproc_t)xdr_remote_domain_interface_addresses_ret, (char *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -7842,30 +7437,23 @@ remoteConnectRegisterCloseCallback(virConnectPtr co= nn, virFreeCallback freecb) { struct private_data *priv =3D conn->privateData; - int ret =3D -1; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (virConnectCloseCallbackDataGetCallback(priv->closeCallback) !=3D N= ULL) { virReportError(VIR_ERR_OPERATION_INVALID, "%s", _("A close callback is already registered")); - goto cleanup; + return -1; } =20 if (priv->serverCloseCallback && call(conn, priv, 0, REMOTE_PROC_CONNECT_REGISTER_CLOSE_CALLBACK, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto cleanup; + return -1; =20 virConnectCloseCallbackDataRegister(priv->closeCallback, conn, cb, opaque, freecb); - ret =3D 0; - - cleanup: - remoteDriverUnlock(priv); - - return ret; + return 0; } =20 static int @@ -7873,29 +7461,22 @@ remoteConnectUnregisterCloseCallback(virConnectPtr = conn, virConnectCloseFunc cb) { struct private_data *priv =3D conn->privateData; - int ret =3D -1; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 if (virConnectCloseCallbackDataGetCallback(priv->closeCallback) !=3D c= b) { virReportError(VIR_ERR_OPERATION_INVALID, "%s", _("A different callback was requested")); - goto cleanup; + return -1; } =20 if (priv->serverCloseCallback && call(conn, priv, 0, REMOTE_PROC_CONNECT_UNREGISTER_CLOSE_CALLBACK, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_void, (char *) NULL) =3D=3D -1) - goto cleanup; + return -1; =20 virConnectCloseCallbackDataUnregister(priv->closeCallback, cb); - ret =3D 0; - - cleanup: - remoteDriverUnlock(priv); - - return ret; + return 0; } =20 static int @@ -7906,11 +7487,10 @@ remoteDomainRename(virDomainPtr dom, const char *ne= w_name, unsigned int flags) remote_domain_rename_args args; remote_domain_rename_ret ret; char *tmp =3D NULL; + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 tmp =3D g_strdup(new_name); =20 - remoteDriverLock(priv); - make_nonnull_domain(&args.dom, dom); args.new_name =3D new_name ? (char **)&new_name : NULL; args.flags =3D flags; @@ -7919,9 +7499,8 @@ remoteDomainRename(virDomainPtr dom, const char *new_= name, unsigned int flags) =20 if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_RENAME, (xdrproc_t)xdr_remote_domain_rename_args, (char *)&args, - (xdrproc_t)xdr_remote_domain_rename_ret, (char *)&ret) =3D=3D= -1) { - goto done; - } + (xdrproc_t)xdr_remote_domain_rename_ret, (char *)&ret) =3D=3D= -1) + return -1; =20 rv =3D ret.retcode; =20 @@ -7930,8 +7509,6 @@ remoteDomainRename(virDomainPtr dom, const char *new_= name, unsigned int flags) dom->name =3D g_steal_pointer(&tmp); } =20 - done: - remoteDriverUnlock(priv); VIR_FREE(tmp); return rv; } @@ -7977,8 +7554,7 @@ remoteNetworkPortGetParameters(virNetworkPortPtr port, struct private_data *priv =3D port->net->conn->privateData; remote_network_port_get_parameters_args args; remote_network_port_get_parameters_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_network_port(&args.port, port); args.flags =3D flags; @@ -7987,7 +7563,7 @@ remoteNetworkPortGetParameters(virNetworkPortPtr port, if (call(port->net->conn, priv, 0, REMOTE_PROC_NETWORK_PORT_GET_PARAME= TERS, (xdrproc_t) xdr_remote_network_port_get_parameters_args, (cha= r *) &args, (xdrproc_t) xdr_remote_network_port_get_parameters_ret, (char= *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -8000,8 +7576,6 @@ remoteNetworkPortGetParameters(virNetworkPortPtr port, =20 cleanup: xdr_free((xdrproc_t) xdr_remote_network_port_get_parameters_ret, (char= *) &ret); - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -8016,8 +7590,7 @@ remoteDomainGetGuestInfo(virDomainPtr dom, struct private_data *priv =3D dom->conn->privateData; remote_domain_get_guest_info_args args; remote_domain_get_guest_info_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); =20 @@ -8029,7 +7602,7 @@ remoteDomainGetGuestInfo(virDomainPtr dom, if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_GUEST_INFO, (xdrproc_t)xdr_remote_domain_get_guest_info_args, (char *)&ar= gs, (xdrproc_t)xdr_remote_domain_get_guest_info_ret, (char *)&ret= ) =3D=3D -1) - goto done; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -8043,9 +7616,6 @@ remoteDomainGetGuestInfo(virDomainPtr dom, cleanup: xdr_free((xdrproc_t)xdr_remote_domain_get_guest_info_ret, (char *) &ret); - - done: - remoteDriverUnlock(priv); return rv; } =20 @@ -8060,8 +7630,7 @@ remoteDomainAuthorizedSSHKeysGet(virDomainPtr domain, struct private_data *priv =3D domain->conn->privateData; remote_domain_authorized_ssh_keys_get_args args; remote_domain_authorized_ssh_keys_get_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.user =3D (char *) user; @@ -8070,9 +7639,8 @@ remoteDomainAuthorizedSSHKeysGet(virDomainPtr domain, =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_AUTHORIZED_SSH_KEYS= _GET, (xdrproc_t) xdr_remote_domain_authorized_ssh_keys_get_args, (= char *)&args, - (xdrproc_t) xdr_remote_domain_authorized_ssh_keys_get_ret, (c= har *)&ret) =3D=3D -1) { - goto cleanup; - } + (xdrproc_t) xdr_remote_domain_authorized_ssh_keys_get_ret, (c= har *)&ret) =3D=3D -1) + return -1; =20 if (ret.keys.keys_len > REMOTE_DOMAIN_AUTHORIZED_SSH_KEYS_MAX) { virReportError(VIR_ERR_RPC, "%s", @@ -8088,7 +7656,6 @@ remoteDomainAuthorizedSSHKeysGet(virDomainPtr domain, rv =3D ret.keys.keys_len; =20 cleanup: - remoteDriverUnlock(priv); xdr_free((xdrproc_t)xdr_remote_domain_authorized_ssh_keys_get_ret, (char *) &ret); return rv; @@ -8137,8 +7704,7 @@ remoteDomainGetMessages(virDomainPtr domain, struct private_data *priv =3D domain->conn->privateData; remote_domain_get_messages_args args; remote_domain_get_messages_ret ret; - - remoteDriverLock(priv); + VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; @@ -8164,7 +7730,6 @@ remoteDomainGetMessages(virDomainPtr domain, rv =3D ret.msgs.msgs_len; =20 cleanup: - remoteDriverUnlock(priv); xdr_free((xdrproc_t)xdr_remote_domain_get_messages_ret, (char *) &ret); return rv; --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358974; cv=none; d=zohomail.com; s=zohoarc; b=bcqv3fmLH4eaYGYYOEbCwkL60r51Mv1PiX5grJwSVpHIqrNFDeUFI4xZiHax32ctO6kcD5u+cvbAPOgmKXQaHZx1LRd0I2J8+O1wyWGj5acoOHVtucIquM0NHPzs2jUjsITLW30cqBwb+KjDyZQv9qu2yDZlSEM+9wivZAxIXDM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358974; h=Content-Type:Content-Transfer-Encoding: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=DrmAugBCQ4jxggG9TBgqjgOUH4oF3DcgML15KhrrAEo=; b=VNBBnGz92UBblThmYQPRtjUekYwM/z4A1Xh0dOQCGjjr8KpVabNJ/p/HJovW4jKmv8zOD5l2Be3mp/6yHC6sowRlTlouc5fiUAfhNU5ey9e9GgslSP7bEDzZpzfodJWmHQK8ZT1PZRMzmKjzTFLi1xU9z1/UmXN90MEI7dZvXFY= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1673358974951922.3066405020317; Tue, 10 Jan 2023 05:56:14 -0800 (PST) Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-543-pXQW9f2YNsmFFbv8h-dHEw-1; Tue, 10 Jan 2023 08:54:33 -0500 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 8784585CBF4; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6828240C2005; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 3F201194E101; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id CFC6D194E114 for ; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id C282940C2064; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id A744740C2004; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358974; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=DrmAugBCQ4jxggG9TBgqjgOUH4oF3DcgML15KhrrAEo=; b=WdPaz19fNcPGry6T68Mx1sXkewHQEJyTli+o68sKa+07FV/4QP/y5nWIJZtHbIwtDZrafq 1Nkup5tvD57jGvps3v5IWooD4xZvVdgJKEJGmyATnv8G5U61VTUPzrSHfHPPxxHJQ0gpVR zdGM01eqMgt9gIdsw+GvsmtaTu28+b8= X-MC-Unique: pXQW9f2YNsmFFbv8h-dHEw-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 6/9] admin: use VIR_LOCK_GUARD in client code Date: Tue, 10 Jan 2023 08:54:20 -0500 Message-Id: <20230110135423.2576907-7-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358976954100005 Using VIR_LOCK_GUARD enables the 'done' goto label to be eliminated. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/admin/admin_remote.c | 72 ++++++++++++---------------------------- 1 file changed, 22 insertions(+), 50 deletions(-) diff --git a/src/admin/admin_remote.c b/src/admin/admin_remote.c index f99b93eef6..5322f087ec 100644 --- a/src/admin/admin_remote.c +++ b/src/admin/admin_remote.c @@ -133,8 +133,7 @@ remoteAdminClientCloseFunc(virNetClient *client G_GNUC_= UNUSED, void *opaque) { virAdmConnectCloseCallbackData *cbdata =3D opaque; - - virObjectLock(cbdata); + VIR_LOCK_GUARD lock =3D virObjectLockGuard(cbdata); =20 if (cbdata->callback) { VIR_DEBUG("Triggering connection close callback %p reason=3D%d, op= aque=3D%p", @@ -142,17 +141,14 @@ remoteAdminClientCloseFunc(virNetClient *client G_GNU= C_UNUSED, cbdata->callback(cbdata->conn, reason, cbdata->opaque); virAdmConnectCloseCallbackDataReset(cbdata); } - virObjectUnlock(cbdata); } =20 static int remoteAdminConnectOpen(virAdmConnectPtr conn, unsigned int flags) { - int rv =3D -1; remoteAdminPriv *priv =3D conn->privateData; admin_connect_open_args args; - - virObjectLock(priv); + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags & ~VIR_CONNECT_NO_ALIASES; =20 @@ -169,40 +165,28 @@ remoteAdminConnectOpen(virAdmConnectPtr conn, unsigne= d int flags) =20 if (call(conn, 0, ADMIN_PROC_CONNECT_OPEN, (xdrproc_t)xdr_admin_connect_open_args, (char *)&args, - (xdrproc_t)xdr_void, (char *)NULL) =3D=3D -1) { - goto done; - } + (xdrproc_t)xdr_void, (char *)NULL) =3D=3D -1) + return -1; =20 - rv =3D 0; - - done: - virObjectUnlock(priv); - return rv; + return 0; } =20 static int remoteAdminConnectClose(virAdmConnectPtr conn) { - int rv =3D -1; remoteAdminPriv *priv =3D conn->privateData; - - virObjectLock(priv); + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 if (call(conn, 0, ADMIN_PROC_CONNECT_CLOSE, (xdrproc_t)xdr_void, (char *)NULL, - (xdrproc_t)xdr_void, (char *)NULL) =3D=3D -1) { - goto done; - } + (xdrproc_t)xdr_void, (char *)NULL) =3D=3D -1) + return -1; =20 virNetClientSetCloseCallback(priv->client, NULL, conn->closeCallback, virObjectUnref); virNetClientClose(priv->client); =20 - rv =3D 0; - - done: - virObjectUnlock(priv); - return rv; + return 0; } =20 static void @@ -249,17 +233,17 @@ remoteAdminServerGetThreadPoolParameters(virAdmServer= Ptr srv, remoteAdminPriv *priv =3D srv->conn->privateData; admin_server_get_threadpool_parameters_args args; admin_server_get_threadpool_parameters_ret ret; + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; make_nonnull_server(&args.srv, srv); =20 memset(&ret, 0, sizeof(ret)); - virObjectLock(priv); =20 if (call(srv->conn, 0, ADMIN_PROC_SERVER_GET_THREADPOOL_PARAMETERS, (xdrproc_t)xdr_admin_server_get_threadpool_parameters_args, (= char *) &args, (xdrproc_t)xdr_admin_server_get_threadpool_parameters_ret, (c= har *) &ret) =3D=3D -1) - goto cleanup; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -272,7 +256,6 @@ remoteAdminServerGetThreadPoolParameters(virAdmServerPt= r srv, =20 cleanup: xdr_free((xdrproc_t)xdr_admin_server_get_threadpool_parameters_ret, (c= har *) &ret); - virObjectUnlock(priv); return rv; } =20 @@ -285,12 +268,11 @@ remoteAdminServerSetThreadPoolParameters(virAdmServer= Ptr srv, int rv =3D -1; remoteAdminPriv *priv =3D srv->conn->privateData; admin_server_set_threadpool_parameters_args args; + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; make_nonnull_server(&args.srv, srv); =20 - virObjectLock(priv); - if (virTypedParamsSerialize(params, nparams, ADMIN_SERVER_THREADPOOL_PARAMETERS_MAX, (struct _virTypedParameterRemote **) &args= .params.params_val, @@ -308,7 +290,6 @@ remoteAdminServerSetThreadPoolParameters(virAdmServerPt= r srv, cleanup: virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); - virObjectUnlock(priv); return rv; } =20 @@ -322,17 +303,17 @@ remoteAdminClientGetInfo(virAdmClientPtr client, remoteAdminPriv *priv =3D client->srv->conn->privateData; admin_client_get_info_args args; admin_client_get_info_ret ret; + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; make_nonnull_client(&args.clnt, client); =20 memset(&ret, 0, sizeof(ret)); - virObjectLock(priv); =20 if (call(client->srv->conn, 0, ADMIN_PROC_CLIENT_GET_INFO, (xdrproc_t)xdr_admin_client_get_info_args, (char *) &args, (xdrproc_t)xdr_admin_client_get_info_ret, (char *) &ret) =3D= =3D -1) - goto cleanup; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -345,7 +326,6 @@ remoteAdminClientGetInfo(virAdmClientPtr client, =20 cleanup: xdr_free((xdrproc_t)xdr_admin_client_get_info_ret, (char *) &ret); - virObjectUnlock(priv); return rv; } =20 @@ -359,18 +339,19 @@ remoteAdminServerGetClientLimits(virAdmServerPtr srv, admin_server_get_client_limits_args args; admin_server_get_client_limits_ret ret; remoteAdminPriv *priv =3D srv->conn->privateData; + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); + args.flags =3D flags; make_nonnull_server(&args.srv, srv); =20 memset(&ret, 0, sizeof(ret)); - virObjectLock(priv); =20 if (call(srv->conn, 0, ADMIN_PROC_SERVER_GET_CLIENT_LIMITS, (xdrproc_t) xdr_admin_server_get_client_limits_args, (char *) &args, (xdrproc_t) xdr_admin_server_get_client_limits_ret, (char *) &ret) =3D=3D -1) - goto cleanup; + return -1; =20 if (virTypedParamsDeserialize((struct _virTypedParameterRemote *) ret.= params.params_val, ret.params.params_len, @@ -384,7 +365,6 @@ remoteAdminServerGetClientLimits(virAdmServerPtr srv, cleanup: xdr_free((xdrproc_t) xdr_admin_server_get_client_limits_ret, (char *) &ret); - virObjectUnlock(priv); return rv; } =20 @@ -397,12 +377,11 @@ remoteAdminServerSetClientLimits(virAdmServerPtr srv, int rv =3D -1; admin_server_set_client_limits_args args; remoteAdminPriv *priv =3D srv->conn->privateData; + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; make_nonnull_server(&args.srv, srv); =20 - virObjectLock(priv); - if (virTypedParamsSerialize(params, nparams, ADMIN_SERVER_CLIENT_LIMITS_MAX, (struct _virTypedParameterRemote **) &args= .params.params_val, @@ -420,7 +399,6 @@ remoteAdminServerSetClientLimits(virAdmServerPtr srv, cleanup: virTypedParamsRemoteFree((struct _virTypedParameterRemote *) args.para= ms.params_val, args.params.params_len); - virObjectUnlock(priv); return rv; } =20 @@ -433,11 +411,11 @@ remoteAdminConnectGetLoggingOutputs(virAdmConnectPtr = conn, remoteAdminPriv *priv =3D conn->privateData; admin_connect_get_logging_outputs_args args; admin_connect_get_logging_outputs_ret ret; + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; =20 memset(&ret, 0, sizeof(ret)); - virObjectLock(priv); =20 if (call(conn, 0, @@ -446,16 +424,13 @@ remoteAdminConnectGetLoggingOutputs(virAdmConnectPtr = conn, (char *) &args, (xdrproc_t) xdr_admin_connect_get_logging_outputs_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (outputs) *outputs =3D g_steal_pointer(&ret.outputs); =20 rv =3D ret.noutputs; xdr_free((xdrproc_t) xdr_admin_connect_get_logging_outputs_ret, (char = *) &ret); - - done: - virObjectUnlock(priv); return rv; } =20 @@ -468,11 +443,11 @@ remoteAdminConnectGetLoggingFilters(virAdmConnectPtr = conn, remoteAdminPriv *priv =3D conn->privateData; admin_connect_get_logging_filters_args args; admin_connect_get_logging_filters_ret ret; + VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; =20 memset(&ret, 0, sizeof(ret)); - virObjectLock(priv); =20 if (call(conn, 0, @@ -481,15 +456,12 @@ remoteAdminConnectGetLoggingFilters(virAdmConnectPtr = conn, (char *) &args, (xdrproc_t) xdr_admin_connect_get_logging_filters_ret, (char *) &ret) =3D=3D -1) - goto done; + return -1; =20 if (filters) *filters =3D ret.filters ? *ret.filters : NULL; =20 rv =3D ret.nfilters; VIR_FREE(ret.filters); - - done: - virObjectUnlock(priv); return rv; } --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358904; cv=none; d=zohomail.com; s=zohoarc; b=BzD3s4Fm88NHmhPWj7XbdWQVkYNLyUfUyEb2I8r5bxPcWXww6gitoDJ80uFycYhhTQO1sqDa/8894onIVpiXlb175I8sT6tOmWKE4Cx7xcVmZCvY+Im23R0UCAwxIkbShiwb/jEEW4Abwweqe962wg4IQAEwjQMiI5huV+QSmJk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358904; h=Content-Type:Content-Transfer-Encoding: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=kBw86mQnEgonPTMbqk3726zIVefNGleyFnbjq4jObkI=; b=mzgdP0tnrxxR8cYO4YDJK1a3hC2rBQYWB5OtPfPigy36sddU5AxGRC/6N0Oa9LTgjGDwsifuX/607amIrhk6qF2o1RDgostNIhYSGZP9KPrpRW3obZwdKrrqbnUT9KFGYQ8AlQ26tzzAneZ1Zqkb597oXR8iiCCLGnvAvgKQxWg= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1673358904475689.3590546103089; Tue, 10 Jan 2023 05:55:04 -0800 (PST) Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-194-J5LWmItiOq28F5xUTIJGxQ-1; Tue, 10 Jan 2023 08:54:34 -0500 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id E4DB8280BCA1; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id CFB0D112131B; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 8EB53194E11B; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 32B55194E101 for ; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 263A640C2064; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id CB56340C2004; Tue, 10 Jan 2023 13:54:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358903; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=kBw86mQnEgonPTMbqk3726zIVefNGleyFnbjq4jObkI=; b=OhJAYEOGpMwua3lpcd23Pdj6GdHN4zO+eW39fsmk7JDPTiBstQDEYGcCS2HpWCR3LE77dY izrs6FfYzWPSmTL9uetqUFBqdjA5en649IZjYfrZznJC3fhoV/uS2J4jB1GtmnDjUooK8j 9lesjqEh329Xv5nlb0dMQdzWCABNJxo= X-MC-Unique: J5LWmItiOq28F5xUTIJGxQ-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 7/9] rpc: use struct zero initializer instead of memset Date: Tue, 10 Jan 2023 08:54:21 -0500 Message-Id: <20230110135423.2576907-8-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.3 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358904695100001 This is a more concise approach and guarantees there is no time window where the struct is uninitialized. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/rpc/gendispatch.pl | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/rpc/gendispatch.pl b/src/rpc/gendispatch.pl index 63d2411474..e9dc8e27d7 100755 --- a/src/rpc/gendispatch.pl +++ b/src/rpc/gendispatch.pl @@ -1534,7 +1534,7 @@ elsif ($mode eq "client") { if ($rettype eq "void") { $call_ret =3D "NULL"; } else { - push(@vars_list, "$rettype ret"); + push(@vars_list, "$rettype ret =3D {0}"); =20 foreach my $ret_member (@{$call->{ret_members}}) { if ($multi_ret) { @@ -1891,11 +1891,6 @@ elsif ($mode eq "client") { print "\n"; } =20 - if ($rettype ne "void") { - print "\n"; - print " memset(&ret, 0, sizeof(ret));\n"; - } - my $callflags =3D "0"; if ($structprefix eq "qemu") { $callflags =3D "REMOTE_CALL_QEMU"; --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358962; cv=none; d=zohomail.com; s=zohoarc; b=WW8fo3lvC3yxdHRVslDTLAbVHTaJmRtBf6Fi709hd151ORLCB9AmgnZYYp61jEsMBNq40ZlsSLbDsNT3tUM5HZeRpxVnQhN/8++HpxTAnhXLUudBlrxJUD+uzvcThe5lMQbR8LntiOsqaGCEzUs/F9OypKVpbjOe53nHi5e8ntA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358962; h=Content-Type:Content-Transfer-Encoding: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=X7vJ4JUjAjOe7+A2wMtjDRufGdJ7Zk/0IErXSqyyo5s=; b=P+PUGMVYkBRwxOyRAAPRkM5/jFEiYAnmOwbdCaQf6gUgQooDra9bPIXCB/WtWAZJqJuMkA+CHKjePTjiLo5zU07ZL6ulLZAq3Zjgjs16HRKVGjpJ4VYe5otY3rRi2SDX5yVu44QdCBKQ7TJY69tdkTgOhUECasG2/oluyG7LufI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1673358962825927.626186421374; Tue, 10 Jan 2023 05:56:02 -0800 (PST) Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-328--wVH9ejAPy6iIvrYux-l5Q-1; Tue, 10 Jan 2023 08:54:36 -0500 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 4158D280BCB1; Tue, 10 Jan 2023 13:54:29 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id 23CCC51E5; Tue, 10 Jan 2023 13:54:29 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id C5437194E110; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 7BF56194F4BD for ; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 5219B40C2064; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id 327BC40C2004; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358961; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=X7vJ4JUjAjOe7+A2wMtjDRufGdJ7Zk/0IErXSqyyo5s=; b=NgxflIkhKrgAr7PiV6wLT4geZl1ORXVzGZOYra0B2TVAoBOZWlTPAMnxoCAwrutToT2ILk LBZ+CO63hPNsFmOxdDzrfNSCna3a9yUZMDS2+ouyZld0JlPn+UkZ36B3wntI+liXO+kyvC Yt8Eqft1fwIKhuUHQRCxhKDj+y/7TnE= X-MC-Unique: -wVH9ejAPy6iIvrYux-l5Q-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 8/9] remote: use struct zero initializer instead of memset Date: Tue, 10 Jan 2023 08:54:22 -0500 Message-Id: <20230110135423.2576907-9-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.5 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358965083100001 This is a more concise approach and guarantees there is no time window where the struct is uninitialized. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/remote/remote_driver.c | 251 +++++++++++-------------------------- 1 file changed, 75 insertions(+), 176 deletions(-) diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index b82e86eff8..206a3829e2 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -1362,7 +1362,7 @@ remoteConnectClose(virConnectPtr conn) static const char * remoteConnectGetType(virConnectPtr conn) { - remote_connect_get_type_ret ret; + remote_connect_get_type_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -1371,7 +1371,6 @@ remoteConnectGetType(virConnectPtr conn) return priv->type; } =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_GET_TYPE, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_remote_connect_get_type_ret, (char *) &ret) = =3D=3D -1) @@ -1384,10 +1383,9 @@ remoteConnectGetType(virConnectPtr conn) static int remoteConnectIsSecure(virConnectPtr conn) { struct private_data *priv =3D conn->privateData; - remote_connect_is_secure_ret ret; + remote_connect_is_secure_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_IS_SECURE, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_remote_connect_is_secure_ret, (char *) &ret) = =3D=3D -1) @@ -1408,10 +1406,9 @@ static int remoteConnectIsEncrypted(virConnectPtr co= nn) { bool encrypted; struct private_data *priv =3D conn->privateData; - remote_connect_is_secure_ret ret; + remote_connect_is_secure_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_IS_SECURE, (xdrproc_t) xdr_void, (char *) NULL, (xdrproc_t) xdr_remote_connect_is_secure_ret, (char *) &ret) = =3D=3D -1) @@ -1438,7 +1435,7 @@ remoteNodeGetCPUStats(virConnectPtr conn, { int rv =3D -1; remote_node_get_cpu_stats_args args; - remote_node_get_cpu_stats_ret ret; + remote_node_get_cpu_stats_ret ret =3D {0}; size_t i; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); @@ -1447,7 +1444,6 @@ remoteNodeGetCPUStats(virConnectPtr conn, args.cpuNum =3D cpuNum; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_CPU_STATS, (xdrproc_t) xdr_remote_node_get_cpu_stats_args, (char *) &args, @@ -1501,7 +1497,7 @@ remoteNodeGetMemoryStats(virConnectPtr conn, { int rv =3D -1; remote_node_get_memory_stats_args args; - remote_node_get_memory_stats_ret ret; + remote_node_get_memory_stats_ret ret =3D {0}; size_t i; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); @@ -1510,7 +1506,6 @@ remoteNodeGetMemoryStats(virConnectPtr conn, args.cellNum =3D cellNum; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_MEMORY_STATS, (xdrproc_t) xdr_remote_node_get_memory_stats_args, (char *) &= args, (xdrproc_t) xdr_remote_node_get_memory_stats_ret, (char *) &r= et) =3D=3D -1) @@ -1560,7 +1555,7 @@ remoteNodeGetCellsFreeMemory(virConnectPtr conn, int maxCells) { remote_node_get_cells_free_memory_args args; - remote_node_get_cells_free_memory_ret ret; + remote_node_get_cells_free_memory_ret ret =3D {0}; size_t i; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); @@ -1575,7 +1570,6 @@ remoteNodeGetCellsFreeMemory(virConnectPtr conn, args.startCell =3D startCell; args.maxcells =3D maxCells; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_CELLS_FREE_MEMORY, (xdrproc_t) xdr_remote_node_get_cells_free_memory_args, (char= *)&args, (xdrproc_t) xdr_remote_node_get_cells_free_memory_ret, (char = *)&ret) =3D=3D -1) @@ -1595,7 +1589,7 @@ remoteConnectListDomains(virConnectPtr conn, int *ids= , int maxids) int rv =3D -1; size_t i; remote_connect_list_domains_args args; - remote_connect_list_domains_ret ret; + remote_connect_list_domains_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -1607,7 +1601,6 @@ remoteConnectListDomains(virConnectPtr conn, int *ids= , int maxids) } args.maxids =3D maxids; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_LIST_DOMAINS, (xdrproc_t) xdr_remote_connect_list_domains_args, (char *) &a= rgs, (xdrproc_t) xdr_remote_connect_list_domains_ret, (char *) &re= t) =3D=3D -1) @@ -1669,7 +1662,7 @@ remoteDomainBlockStatsFlags(virDomainPtr domain, { int rv =3D -1; remote_domain_block_stats_flags_args args; - remote_domain_block_stats_flags_ret ret; + remote_domain_block_stats_flags_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -1678,7 +1671,6 @@ remoteDomainBlockStatsFlags(virDomainPtr domain, args.path =3D (char *) path; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_BLOCK_STATS_FLAGS, (xdrproc_t) xdr_remote_domain_block_stats_flags_args, (char *= ) &args, (xdrproc_t) xdr_remote_domain_block_stats_flags_ret, (char *)= &ret) =3D=3D -1) @@ -1727,7 +1719,7 @@ remoteDomainGetMemoryParameters(virDomainPtr domain, { int rv =3D -1; remote_domain_get_memory_parameters_args args; - remote_domain_get_memory_parameters_ret ret; + remote_domain_get_memory_parameters_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -1735,7 +1727,6 @@ remoteDomainGetMemoryParameters(virDomainPtr domain, args.nparams =3D *nparams; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_MEMORY_PARAMETE= RS, (xdrproc_t) xdr_remote_domain_get_memory_parameters_args, (ch= ar *) &args, (xdrproc_t) xdr_remote_domain_get_memory_parameters_ret, (cha= r *) &ret) =3D=3D -1) @@ -1772,7 +1763,7 @@ remoteDomainGetNumaParameters(virDomainPtr domain, { int rv =3D -1; remote_domain_get_numa_parameters_args args; - remote_domain_get_numa_parameters_ret ret; + remote_domain_get_numa_parameters_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -1780,7 +1771,6 @@ remoteDomainGetNumaParameters(virDomainPtr domain, args.nparams =3D *nparams; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_NUMA_PARAMETERS, (xdrproc_t) xdr_remote_domain_get_numa_parameters_args, (char= *) &args, (xdrproc_t) xdr_remote_domain_get_numa_parameters_ret, (char = *) &ret) =3D=3D -1) @@ -1818,14 +1808,13 @@ remoteDomainGetLaunchSecurityInfo(virDomainPtr doma= in, { int rv =3D -1; remote_domain_get_launch_security_info_args args; - remote_domain_get_launch_security_info_ret ret; + remote_domain_get_launch_security_info_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_LAUNCH_SECURITY= _INFO, (xdrproc_t) xdr_remote_domain_get_launch_security_info_args, = (char *) &args, (xdrproc_t) xdr_remote_domain_get_launch_security_info_ret, (= char *) &ret) =3D=3D -1) @@ -1854,14 +1843,13 @@ remoteDomainGetPerfEvents(virDomainPtr domain, { int rv =3D -1; remote_domain_get_perf_events_args args; - remote_domain_get_perf_events_ret ret; + remote_domain_get_perf_events_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_PERF_EVENTS, (xdrproc_t) xdr_remote_domain_get_perf_events_args, (char *) = &args, (xdrproc_t) xdr_remote_domain_get_perf_events_ret, (char *) &= ret) =3D=3D -1) @@ -1889,7 +1877,7 @@ remoteDomainGetBlkioParameters(virDomainPtr domain, { int rv =3D -1; remote_domain_get_blkio_parameters_args args; - remote_domain_get_blkio_parameters_ret ret; + remote_domain_get_blkio_parameters_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -1897,7 +1885,6 @@ remoteDomainGetBlkioParameters(virDomainPtr domain, args.nparams =3D *nparams; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_BLKIO_PARAMETER= S, (xdrproc_t) xdr_remote_domain_get_blkio_parameters_args, (cha= r *) &args, (xdrproc_t) xdr_remote_domain_get_blkio_parameters_ret, (char= *) &ret) =3D=3D -1) @@ -1937,7 +1924,7 @@ remoteDomainGetVcpuPinInfo(virDomainPtr domain, int rv =3D -1; size_t i; remote_domain_get_vcpu_pin_info_args args; - remote_domain_get_vcpu_pin_info_ret ret; + remote_domain_get_vcpu_pin_info_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -1961,8 +1948,6 @@ remoteDomainGetVcpuPinInfo(virDomainPtr domain, args.maplen =3D maplen; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_VCPU_PIN_INFO, (xdrproc_t) xdr_remote_domain_get_vcpu_pin_info_args, (char *) &args, @@ -2037,7 +2022,7 @@ remoteDomainGetEmulatorPinInfo(virDomainPtr domain, int rv =3D -1; size_t i; remote_domain_get_emulator_pin_info_args args; - remote_domain_get_emulator_pin_info_ret ret; + remote_domain_get_emulator_pin_info_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2053,8 +2038,6 @@ remoteDomainGetEmulatorPinInfo(virDomainPtr domain, args.maplen =3D maplen; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_EMULATOR_PIN_IN= FO, (xdrproc_t) xdr_remote_domain_get_emulator_pin_info_args, (char *) &args, @@ -2092,7 +2075,7 @@ remoteDomainGetVcpus(virDomainPtr domain, int rv =3D -1; size_t i; remote_domain_get_vcpus_args args; - remote_domain_get_vcpus_ret ret; + remote_domain_get_vcpus_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2114,7 +2097,6 @@ remoteDomainGetVcpus(virDomainPtr domain, args.maxinfo =3D maxinfo; args.maplen =3D maplen; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_VCPUS, (xdrproc_t) xdr_remote_domain_get_vcpus_args, (char *) &args, (xdrproc_t) xdr_remote_domain_get_vcpus_ret, (char *) &ret) = =3D=3D -1) @@ -2162,7 +2144,7 @@ remoteDomainGetIOThreadInfo(virDomainPtr dom, size_t i; struct private_data *priv =3D dom->conn->privateData; remote_domain_get_iothread_info_args args; - remote_domain_get_iothread_info_ret ret; + remote_domain_get_iothread_info_ret ret =3D {0}; remote_domain_iothread_info *src; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2170,8 +2152,6 @@ remoteDomainGetIOThreadInfo(virDomainPtr dom, =20 args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_IOTHREAD_INFO, (xdrproc_t)xdr_remote_domain_get_iothread_info_args, (char *)&args, @@ -2223,13 +2203,12 @@ static int remoteDomainGetSecurityLabel(virDomainPtr domain, virSecurityLabelPtr secl= abel) { remote_domain_get_security_label_args args; - remote_domain_get_security_label_ret ret; + remote_domain_get_security_label_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; int rv =3D -1; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); - memset(&ret, 0, sizeof(ret)); memset(seclabel, 0, sizeof(*seclabel)); =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL, @@ -2257,14 +2236,13 @@ static int remoteDomainGetSecurityLabelList(virDomainPtr domain, virSecurityLabelPtr*= seclabels) { remote_domain_get_security_label_list_args args; - remote_domain_get_security_label_list_ret ret; + remote_domain_get_security_label_list_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; size_t i; int rv =3D -1; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); - memset(&ret, 0, sizeof(ret)); =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_SECURITY_LABEL_= LIST, (xdrproc_t) xdr_remote_domain_get_security_label_list_args, (= char *)&args, @@ -2299,14 +2277,13 @@ remoteDomainGetState(virDomainPtr domain, unsigned int flags) { remote_domain_get_state_args args; - remote_domain_get_state_ret ret; + remote_domain_get_state_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_STATE, (xdrproc_t) xdr_remote_domain_get_state_args, (char *) &args, (xdrproc_t) xdr_remote_domain_get_state_ret, (char *) &ret) = =3D=3D -1) @@ -2322,12 +2299,11 @@ remoteDomainGetState(virDomainPtr domain, static int remoteNodeGetSecurityModel(virConnectPtr conn, virSecurityModelPtr secmode= l) { - remote_node_get_security_model_ret ret; + remote_node_get_security_model_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; int rv =3D -1; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); memset(secmodel, 0, sizeof(*secmodel)); =20 if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_SECURITY_MODEL, @@ -2366,7 +2342,7 @@ remoteDomainMigratePrepare(virConnectPtr dconn, unsigned long resource) { remote_domain_migrate_prepare_args args; - remote_domain_migrate_prepare_ret ret; + remote_domain_migrate_prepare_ret ret =3D {0}; struct private_data *priv =3D dconn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2375,7 +2351,6 @@ remoteDomainMigratePrepare(virConnectPtr dconn, args.dname =3D dname =3D=3D NULL ? NULL : (char **) &dname; args.resource =3D resource; =20 - memset(&ret, 0, sizeof(ret)); if (call(dconn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE, (xdrproc_t) xdr_remote_domain_migrate_prepare_args, (char *) = &args, (xdrproc_t) xdr_remote_domain_migrate_prepare_ret, (char *) &= ret) =3D=3D -1) @@ -2402,7 +2377,7 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, { int rv =3D -1; remote_domain_migrate_prepare2_args args; - remote_domain_migrate_prepare2_ret ret; + remote_domain_migrate_prepare2_ret ret =3D {0}; struct private_data *priv =3D dconn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2412,7 +2387,6 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, args.resource =3D resource; args.dom_xml =3D (char *) dom_xml; =20 - memset(&ret, 0, sizeof(ret)); if (call(dconn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE2, (xdrproc_t) xdr_remote_domain_migrate_prepare2_args, (char *)= &args, (xdrproc_t) xdr_remote_domain_migrate_prepare2_ret, (char *) = &ret) =3D=3D -1) @@ -2470,7 +2444,6 @@ remoteDomainCreate(virDomainPtr domain) * it returned. */ memcpy(args2.uuid, domain->uuid, VIR_UUID_BUFLEN); - memset(&ret2, 0, sizeof(ret2)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_LOOKUP_BY_UUID, (xdrproc_t) xdr_remote_domain_lookup_by_uuid_args, (char *) &= args2, (xdrproc_t) xdr_remote_domain_lookup_by_uuid_ret, (char *) &r= et2) =3D=3D -1) @@ -2486,13 +2459,12 @@ static char * remoteDomainGetSchedulerType(virDomainPtr domain, int *nparams) { remote_domain_get_scheduler_type_args args; - remote_domain_get_scheduler_type_ret ret; + remote_domain_get_scheduler_type_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_SCHEDULER_TYPE, (xdrproc_t) xdr_remote_domain_get_scheduler_type_args, (char = *) &args, (xdrproc_t) xdr_remote_domain_get_scheduler_type_ret, (char *= ) &ret) =3D=3D -1) @@ -2512,7 +2484,7 @@ remoteDomainMemoryStats(virDomainPtr domain, { int rv =3D -1; remote_domain_memory_stats_args args; - remote_domain_memory_stats_ret ret; + remote_domain_memory_stats_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; size_t i; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); @@ -2526,7 +2498,6 @@ remoteDomainMemoryStats(virDomainPtr domain, } args.maxStats =3D nr_stats; args.flags =3D flags; - memset(&ret, 0, sizeof(ret)); =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_MEMORY_STATS, (xdrproc_t) xdr_remote_domain_memory_stats_args, @@ -2555,7 +2526,7 @@ remoteDomainBlockPeek(virDomainPtr domain, { int rv =3D -1; remote_domain_block_peek_args args; - remote_domain_block_peek_ret ret; + remote_domain_block_peek_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2572,7 +2543,6 @@ remoteDomainBlockPeek(virDomainPtr domain, args.size =3D size; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_BLOCK_PEEK, (xdrproc_t) xdr_remote_domain_block_peek_args, (char *) &args, @@ -2603,7 +2573,7 @@ remoteDomainMemoryPeek(virDomainPtr domain, { int rv =3D -1; remote_domain_memory_peek_args args; - remote_domain_memory_peek_ret ret; + remote_domain_memory_peek_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2619,7 +2589,6 @@ remoteDomainMemoryPeek(virDomainPtr domain, args.size =3D size; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_MEMORY_PEEK, (xdrproc_t) xdr_remote_domain_memory_peek_args, (char *) &args, @@ -2648,7 +2617,7 @@ static int remoteDomainGetBlockJobInfo(virDomainPtr d= omain, unsigned int flags) { remote_domain_get_block_job_info_args args; - remote_domain_get_block_job_info_ret ret; + remote_domain_get_block_job_info_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2682,7 +2651,7 @@ static int remoteDomainGetBlockIoTune(virDomainPtr do= main, { int rv =3D -1; remote_domain_get_block_io_tune_args args; - remote_domain_get_block_io_tune_ret ret; + remote_domain_get_block_io_tune_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -2691,9 +2660,6 @@ static int remoteDomainGetBlockIoTune(virDomainPtr do= main, args.nparams =3D *nparams; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - - if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_BLOCK_IO_TUNE, (xdrproc_t) xdr_remote_domain_get_block_io_tune_args, (char *) &args, @@ -2735,7 +2701,7 @@ static int remoteDomainGetCPUStats(virDomainPtr domai= n, { struct private_data *priv =3D domain->conn->privateData; remote_domain_get_cpu_stats_args args; - remote_domain_get_cpu_stats_ret ret; + remote_domain_get_cpu_stats_ret ret =3D {0}; int rv =3D -1; int cpu; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); @@ -2759,8 +2725,6 @@ static int remoteDomainGetCPUStats(virDomainPtr domai= n, args.ncpus =3D ncpus; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_CPU_STATS, (xdrproc_t) xdr_remote_domain_get_cpu_stats_args, (char *) &args, @@ -2826,7 +2790,7 @@ remoteConnectNetworkEventRegisterAny(virConnectPtr co= nn, { struct private_data *priv =3D conn->privateData; remote_connect_network_event_register_any_args args; - remote_connect_network_event_register_any_ret ret; + remote_connect_network_event_register_any_ret ret =3D {0}; int callbackID; int count; remote_nonnull_network network; @@ -2849,7 +2813,6 @@ remoteConnectNetworkEventRegisterAny(virConnectPtr co= nn, args.net =3D NULL; } =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_NETWORK_EVENT_REGISTER= _ANY, (xdrproc_t) xdr_remote_connect_network_event_register_any= _args, (char *) &args, (xdrproc_t) xdr_remote_connect_network_event_register_any= _ret, (char *) &ret) =3D=3D -1) { @@ -2908,7 +2871,7 @@ remoteConnectStoragePoolEventRegisterAny(virConnectPt= r conn, { struct private_data *priv =3D conn->privateData; remote_connect_storage_pool_event_register_any_args args; - remote_connect_storage_pool_event_register_any_ret ret; + remote_connect_storage_pool_event_register_any_ret ret =3D {0}; int callbackID; int count; remote_nonnull_storage_pool storage_pool; @@ -2931,7 +2894,6 @@ remoteConnectStoragePoolEventRegisterAny(virConnectPt= r conn, args.pool =3D NULL; } =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_STORAGE_POOL_EVENT_REG= ISTER_ANY, (xdrproc_t) xdr_remote_connect_storage_pool_event_registe= r_any_args, (char *) &args, (xdrproc_t) xdr_remote_connect_storage_pool_event_registe= r_any_ret, (char *) &ret) =3D=3D -1) { @@ -2992,7 +2954,7 @@ remoteConnectNodeDeviceEventRegisterAny(virConnectPtr= conn, { struct private_data *priv =3D conn->privateData; remote_connect_node_device_event_register_any_args args; - remote_connect_node_device_event_register_any_ret ret; + remote_connect_node_device_event_register_any_ret ret =3D {0}; int callbackID; int count; remote_nonnull_node_device node_device; @@ -3015,7 +2977,6 @@ remoteConnectNodeDeviceEventRegisterAny(virConnectPtr= conn, args.dev =3D NULL; } =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_NODE_DEVICE_EVENT_REGI= STER_ANY, (xdrproc_t) xdr_remote_connect_node_device_event_register= _any_args, (char *) &args, (xdrproc_t) xdr_remote_connect_node_device_event_register= _any_ret, (char *) &ret) =3D=3D -1) { @@ -3077,7 +3038,7 @@ remoteConnectSecretEventRegisterAny(virConnectPtr con= n, { struct private_data *priv =3D conn->privateData; remote_connect_secret_event_register_any_args args; - remote_connect_secret_event_register_any_ret ret; + remote_connect_secret_event_register_any_ret ret =3D {0}; int callbackID; int count; remote_nonnull_secret sec; @@ -3100,7 +3061,6 @@ remoteConnectSecretEventRegisterAny(virConnectPtr con= n, args.secret =3D NULL; } =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_SECRET_EVENT_REGISTER_= ANY, (xdrproc_t) xdr_remote_connect_secret_event_register_any_= args, (char *) &args, (xdrproc_t) xdr_remote_connect_secret_event_register_any_= ret, (char *) &ret) =3D=3D -1) { @@ -3163,7 +3123,7 @@ remoteConnectDomainQemuMonitorEventRegister(virConnec= tPtr conn, { struct private_data *priv =3D conn->privateData; qemu_connect_domain_monitor_event_register_args args; - qemu_connect_domain_monitor_event_register_ret ret; + qemu_connect_domain_monitor_event_register_ret ret =3D {0}; int callbackID; int count; remote_nonnull_domain domain; @@ -3188,7 +3148,6 @@ remoteConnectDomainQemuMonitorEventRegister(virConnec= tPtr conn, args.event =3D event ? (char **) &event : NULL; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, REMOTE_CALL_QEMU, QEMU_PROC_CONNECT_DOMAIN_MO= NITOR_EVENT_REGISTER, (xdrproc_t) xdr_qemu_connect_domain_monitor_event_registe= r_args, (char *) &args, (xdrproc_t) xdr_qemu_connect_domain_monitor_event_registe= r_ret, (char *) &ret) =3D=3D -1) { @@ -3246,7 +3205,7 @@ remoteConnectFindStoragePoolSources(virConnectPtr con= n, { char *rv =3D NULL; remote_connect_find_storage_pool_sources_args args; - remote_connect_find_storage_pool_sources_ret ret; + remote_connect_find_storage_pool_sources_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -3254,7 +3213,6 @@ remoteConnectFindStoragePoolSources(virConnectPtr con= n, args.srcSpec =3D srcSpec ? (char **)&srcSpec : NULL; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_FIND_STORAGE_POOL_SOURCES, (xdrproc_t) xdr_remote_connect_find_storage_pool_sources_args= , (char *) &args, (xdrproc_t) xdr_remote_connect_find_storage_pool_sources_ret,= (char *) &ret) =3D=3D -1) @@ -3350,10 +3308,9 @@ remoteAuthenticate(virConnectPtr conn, struct privat= e_data *priv, virConnectAuthPtr auth G_GNUC_UNUSED, const char *authtype) { - struct remote_auth_list_ret ret; + struct remote_auth_list_ret ret =3D {0}; int err, type =3D REMOTE_AUTH_NONE; =20 - memset(&ret, 0, sizeof(ret)); err =3D call(conn, priv, 0, REMOTE_PROC_AUTH_LIST, (xdrproc_t) xdr_void, (char *) NULL, @@ -3981,10 +3938,9 @@ static int remoteAuthPolkit(virConnectPtr conn, struct private_data *priv, virConnectAuthPtr auth G_GNUC_UNUSED) { - remote_auth_polkit_ret ret; + remote_auth_polkit_ret ret =3D {0}; VIR_DEBUG("Client initialize PolicyKit authentication"); =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_AUTH_POLKIT, (xdrproc_t) xdr_void, (char *)NULL, (xdrproc_t) xdr_remote_auth_polkit_ret, (char *) &ret) !=3D 0= ) { @@ -4021,12 +3977,11 @@ remoteConnectDomainEventRegister(virConnectPtr conn, /* Tell the server when we are the first callback registering */ if (priv->serverEventFilter) { remote_connect_domain_event_callback_register_any_args args; - remote_connect_domain_event_callback_register_any_ret ret; + remote_connect_domain_event_callback_register_any_ret ret =3D = {0}; =20 args.eventID =3D VIR_DOMAIN_EVENT_ID_LIFECYCLE; args.dom =3D NULL; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_DOMAIN_EVENT_CALLBACK_REGISTER_AN= Y, (xdrproc_t) xdr_remote_connect_domain_event_callback_= register_any_args, (char *) &args, @@ -5131,13 +5086,12 @@ remoteSecretGetValue(virSecretPtr secret, size_t *v= alue_size, unsigned int flags) { remote_secret_get_value_args args; - remote_secret_get_value_ret ret; + remote_secret_get_value_ret ret =3D {0}; struct private_data *priv =3D secret->conn->privateData; =20 make_nonnull_secret(&args.secret, secret); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(secret->conn, priv, 0, REMOTE_PROC_SECRET_GET_VALUE, (xdrproc_t) xdr_remote_secret_get_value_args, (char *) &args, (xdrproc_t) xdr_remote_secret_get_value_ret, (char *) &ret) = =3D=3D -1) @@ -5538,7 +5492,7 @@ remoteConnectDomainEventRegisterAny(virConnectPtr con= n, if (count =3D=3D 1) { if (priv->serverEventFilter) { remote_connect_domain_event_callback_register_any_args args; - remote_connect_domain_event_callback_register_any_ret ret; + remote_connect_domain_event_callback_register_any_ret ret =3D = {0}; =20 args.eventID =3D eventID; if (dom) { @@ -5548,7 +5502,6 @@ remoteConnectDomainEventRegisterAny(virConnectPtr con= n, args.dom =3D NULL; } =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_DOMAIN_EVENT_CALLB= ACK_REGISTER_ANY, (xdrproc_t) xdr_remote_connect_domain_event_callback_= register_any_args, (char *) &args, (xdrproc_t) xdr_remote_connect_domain_event_callback_= register_any_ret, (char *) &ret) =3D=3D -1) { @@ -5632,7 +5585,7 @@ remoteDomainQemuMonitorCommand(virDomainPtr domain, c= onst char *cmd, { int rv =3D -1; qemu_domain_monitor_command_args args; - qemu_domain_monitor_command_ret ret; + qemu_domain_monitor_command_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -5640,7 +5593,6 @@ remoteDomainQemuMonitorCommand(virDomainPtr domain, c= onst char *cmd, args.cmd =3D (char *)cmd; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, REMOTE_CALL_QEMU, QEMU_PROC_DOMAIN_MONITO= R_COMMAND, (xdrproc_t) xdr_qemu_domain_monitor_command_args, (char *) &a= rgs, (xdrproc_t) xdr_qemu_domain_monitor_command_ret, (char *) &re= t) =3D=3D -1) @@ -5667,7 +5619,7 @@ remoteDomainQemuMonitorCommandWithFiles(virDomainPtr = domain, unsigned int flags) { qemu_domain_monitor_command_with_files_args args; - qemu_domain_monitor_command_with_files_ret ret; + qemu_domain_monitor_command_with_files_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; size_t rpc_noutfiles =3D 0; g_autofree int *rpc_outfiles =3D NULL; @@ -5678,7 +5630,6 @@ remoteDomainQemuMonitorCommandWithFiles(virDomainPtr = domain, args.cmd =3D (char *)cmd; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (callFull(domain->conn, priv, REMOTE_CALL_QEMU, infiles, ninfiles, &rpc_outfiles, &rpc_noutfiles, QEMU_PROC_DOMAIN_MONITOR_COMMAND_WITH_FILES, @@ -5717,12 +5668,10 @@ remoteDomainMigrateBegin3(virDomainPtr domain, { char *rv =3D NULL; remote_domain_migrate_begin3_args args; - remote_domain_migrate_begin3_ret ret; + remote_domain_migrate_begin3_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - make_nonnull_domain(&args.dom, domain); args.xmlin =3D xmlin =3D=3D NULL ? NULL : (char **) &xmlin; args.flags =3D flags; @@ -5770,12 +5719,10 @@ remoteDomainMigratePrepare3(virConnectPtr dconn, { int rv =3D -1; remote_domain_migrate_prepare3_args args; - remote_domain_migrate_prepare3_ret ret; + remote_domain_migrate_prepare3_ret ret =3D {0}; struct private_data *priv =3D dconn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - args.cookie_in.cookie_in_val =3D (char *)cookiein; args.cookie_in.cookie_in_len =3D cookieinlen; args.uri_in =3D uri_in =3D=3D NULL ? NULL : (char **) &uri_in; @@ -5784,7 +5731,6 @@ remoteDomainMigratePrepare3(virConnectPtr dconn, args.resource =3D resource; args.dom_xml =3D (char *) dom_xml; =20 - memset(&ret, 0, sizeof(ret)); if (call(dconn, priv, 0, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE3, (xdrproc_t) xdr_remote_domain_migrate_prepare3_args, (char *)= &args, (xdrproc_t) xdr_remote_domain_migrate_prepare3_ret, (char *) = &ret) =3D=3D -1) @@ -5835,12 +5781,10 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dco= nn, { struct private_data *priv =3D dconn->privateData; remote_domain_migrate_prepare_tunnel3_args args; - remote_domain_migrate_prepare_tunnel3_ret ret; + remote_domain_migrate_prepare_tunnel3_ret ret =3D {0}; virNetClientStream *netst; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - if (!(netst =3D virNetClientStreamNew(priv->remoteProgram, REMOTE_PROC_DOMAIN_MIGRATE_PREPARE= _TUNNEL3, priv->counter, @@ -5903,12 +5847,10 @@ remoteDomainMigratePerform3(virDomainPtr dom, unsigned long resource) { remote_domain_migrate_perform3_args args; - remote_domain_migrate_perform3_ret ret; + remote_domain_migrate_perform3_ret ret =3D {0}; struct private_data *priv =3D dom->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - make_nonnull_domain(&args.dom, dom); =20 args.xmlin =3D xmlin =3D=3D NULL ? NULL : (char **) &xmlin; @@ -5956,13 +5898,11 @@ remoteDomainMigrateFinish3(virConnectPtr dconn, int cancelled) { remote_domain_migrate_finish3_args args; - remote_domain_migrate_finish3_ret ret; + remote_domain_migrate_finish3_ret ret =3D {0}; struct private_data *priv =3D dconn->privateData; virDomainPtr rv =3D NULL; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - args.cookie_in.cookie_in_val =3D (char *)cookiein; args.cookie_in.cookie_in_len =3D cookieinlen; args.dname =3D (char *) dname; @@ -6035,7 +5975,7 @@ remoteConnectGetCPUModelNames(virConnectPtr conn, size_t i; g_auto(GStrv) retmodels =3D NULL; remote_connect_get_cpu_model_names_args args; - remote_connect_get_cpu_model_names_ret ret; + remote_connect_get_cpu_model_names_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -6043,7 +5983,6 @@ remoteConnectGetCPUModelNames(virConnectPtr conn, args.need_results =3D !!models; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_CONNECT_GET_CPU_MODEL_NAMES, (xdrproc_t) xdr_remote_connect_get_cpu_model_names_args, (char *) &args, @@ -6200,15 +6139,13 @@ remoteDomainGetDiskErrors(virDomainPtr dom, int rv =3D -1; struct private_data *priv =3D dom->conn->privateData; remote_domain_get_disk_errors_args args; - remote_domain_get_disk_errors_ret ret; + remote_domain_get_disk_errors_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); args.maxerrors =3D maxerrors; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_DISK_ERRORS, (xdrproc_t) xdr_remote_domain_get_disk_errors_args, (char *) &args, @@ -6307,7 +6244,7 @@ remoteDomainGetInterfaceParameters(virDomainPtr domai= n, { int rv =3D -1; remote_domain_get_interface_parameters_args args; - remote_domain_get_interface_parameters_ret ret; + remote_domain_get_interface_parameters_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -6316,7 +6253,6 @@ remoteDomainGetInterfaceParameters(virDomainPtr domai= n, args.nparams =3D *nparams; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_INTERFACE_PARAM= ETERS, (xdrproc_t) xdr_remote_domain_get_interface_parameters_args, = (char *) &args, (xdrproc_t) xdr_remote_domain_get_interface_parameters_ret, (= char *) &ret) =3D=3D -1) @@ -6355,14 +6291,13 @@ remoteNodeGetMemoryParameters(virConnectPtr conn, { int rv =3D -1; remote_node_get_memory_parameters_args args; - remote_node_get_memory_parameters_ret ret; + remote_node_get_memory_parameters_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.nparams =3D *nparams; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_MEMORY_PARAMETERS, (xdrproc_t) xdr_remote_node_get_memory_parameters_args, (char= *) &args, (xdrproc_t) xdr_remote_node_get_memory_parameters_ret, (char = *) &ret) =3D=3D -1) @@ -6401,13 +6336,12 @@ remoteNodeGetSEVInfo(virConnectPtr conn, { int rv =3D -1; remote_node_get_sev_info_args args; - remote_node_get_sev_info_ret ret; + remote_node_get_sev_info_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_SEV_INFO, (xdrproc_t) xdr_remote_node_get_sev_info_args, (char *) &args, (xdrproc_t) xdr_remote_node_get_sev_info_ret, (char *) &ret) = =3D=3D -1) @@ -6436,7 +6370,7 @@ remoteNodeGetCPUMap(virConnectPtr conn, { int rv =3D -1; remote_node_get_cpu_map_args args; - remote_node_get_cpu_map_ret ret; + remote_node_get_cpu_map_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -6444,7 +6378,6 @@ remoteNodeGetCPUMap(virConnectPtr conn, args.need_online =3D !!online; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_CPU_MAP, (xdrproc_t) xdr_remote_node_get_cpu_map_args, (char *) &args, @@ -6506,14 +6439,13 @@ remoteDomainGetJobStats(virDomainPtr domain, { int rv =3D -1; remote_domain_get_job_stats_args args; - remote_domain_get_job_stats_ret ret; + remote_domain_get_job_stats_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_JOB_STATS, (xdrproc_t) xdr_remote_domain_get_job_stats_args, (char *) &a= rgs, (xdrproc_t) xdr_remote_domain_get_job_stats_ret, (char *) &re= t) =3D=3D -1) @@ -6546,12 +6478,10 @@ remoteDomainMigrateBegin3Params(virDomainPtr domain, { char *rv =3D NULL; remote_domain_migrate_begin3_params_args args; - remote_domain_migrate_begin3_params_ret ret; + remote_domain_migrate_begin3_params_ret ret =3D {0}; struct private_data *priv =3D domain->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - make_nonnull_domain(&args.dom, domain); args.flags =3D flags; =20 @@ -6606,12 +6536,10 @@ remoteDomainMigratePrepare3Params(virConnectPtr dco= nn, { int rv =3D -1; remote_domain_migrate_prepare3_params_args args; - remote_domain_migrate_prepare3_params_ret ret; + remote_domain_migrate_prepare3_params_ret ret =3D {0}; struct private_data *priv =3D dconn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - if (virTypedParamsSerialize(params, nparams, REMOTE_DOMAIN_MIGRATE_PARAM_LIST_MAX, (struct _virTypedParameterRemote **) &args= .params.params_val, @@ -6679,12 +6607,10 @@ remoteDomainMigratePrepareTunnel3Params(virConnectP= tr dconn, struct private_data *priv =3D dconn->privateData; int rv =3D -1; remote_domain_migrate_prepare_tunnel3_params_args args; - remote_domain_migrate_prepare_tunnel3_params_ret ret; + remote_domain_migrate_prepare_tunnel3_params_ret ret =3D {0}; virNetClientStream *netst; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - args.cookie_in.cookie_in_val =3D (char *)cookiein; args.cookie_in.cookie_in_len =3D cookieinlen; args.flags =3D flags; @@ -6758,12 +6684,10 @@ remoteDomainMigratePerform3Params(virDomainPtr dom, { int rv =3D -1; remote_domain_migrate_perform3_params_args args; - remote_domain_migrate_perform3_params_ret ret; + remote_domain_migrate_perform3_params_ret ret =3D {0}; struct private_data *priv =3D dom->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - make_nonnull_domain(&args.dom, dom); args.dconnuri =3D dconnuri =3D=3D NULL ? NULL : (char **) &dconnuri; args.cookie_in.cookie_in_val =3D (char *)cookiein; @@ -6820,13 +6744,11 @@ remoteDomainMigrateFinish3Params(virConnectPtr dcon= n, int cancelled) { remote_domain_migrate_finish3_params_args args; - remote_domain_migrate_finish3_params_ret ret; + remote_domain_migrate_finish3_params_ret ret =3D {0}; struct private_data *priv =3D dconn->privateData; virDomainPtr rv =3D NULL; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 - memset(&ret, 0, sizeof(ret)); - args.cookie_in.cookie_in_val =3D (char *)cookiein; args.cookie_in.cookie_in_len =3D cookieinlen; args.flags =3D flags; @@ -6923,14 +6845,12 @@ remoteDomainCreateXMLWithFiles(virConnectPtr conn, = const char *xml_desc, virDomainPtr rv =3D NULL; struct private_data *priv =3D conn->privateData; remote_domain_create_xml_with_files_args args; - remote_domain_create_xml_with_files_ret ret; + remote_domain_create_xml_with_files_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 args.xml_desc =3D (char *)xml_desc; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (callFull(conn, priv, 0, files, nfiles, NULL, NULL, @@ -6953,14 +6873,12 @@ remoteDomainCreateWithFiles(virDomainPtr dom, { struct private_data *priv =3D dom->conn->privateData; remote_domain_create_with_files_args args; - remote_domain_create_with_files_ret ret; + remote_domain_create_with_files_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (callFull(dom->conn, priv, 0, files, nfiles, NULL, NULL, @@ -6982,7 +6900,7 @@ remoteDomainGetTime(virDomainPtr dom, { struct private_data *priv =3D dom->conn->privateData; remote_domain_get_time_args args; - remote_domain_get_time_ret ret; + remote_domain_get_time_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); @@ -7012,7 +6930,7 @@ remoteNodeGetFreePages(virConnectPtr conn, unsigned int flags) { remote_node_get_free_pages_args args; - remote_node_get_free_pages_ret ret; + remote_node_get_free_pages_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -7029,7 +6947,6 @@ remoteNodeGetFreePages(virConnectPtr conn, args.cellCount =3D cellCount; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_GET_FREE_PAGES, (xdrproc_t) xdr_remote_node_get_free_pages_args, (char *)&arg= s, (xdrproc_t) xdr_remote_node_get_free_pages_ret, (char *)&ret)= =3D=3D -1) @@ -7093,7 +7010,7 @@ remoteNetworkGetDHCPLeases(virNetworkPtr net, size_t i; struct private_data *priv =3D net->conn->privateData; remote_network_get_dhcp_leases_args args; - remote_network_get_dhcp_leases_ret ret; + remote_network_get_dhcp_leases_ret ret =3D {0}; virNetworkDHCPLeasePtr *leases_ret =3D NULL; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -7102,8 +7019,6 @@ remoteNetworkGetDHCPLeases(virNetworkPtr net, args.flags =3D flags; args.need_results =3D !!leases; =20 - memset(&ret, 0, sizeof(ret)); - if (call(net->conn, priv, 0, REMOTE_PROC_NETWORK_GET_DHCP_LEASES, (xdrproc_t)xdr_remote_network_get_dhcp_leases_args, (char *)&= args, (xdrproc_t)xdr_remote_network_get_dhcp_leases_ret, (char *)&r= et) =3D=3D -1) @@ -7157,7 +7072,7 @@ remoteConnectGetAllDomainStats(virConnectPtr conn, int rv =3D -1; size_t i; remote_connect_get_all_domain_stats_args args; - remote_connect_get_all_domain_stats_ret ret; + remote_connect_get_all_domain_stats_ret ret =3D {0}; virDomainStatsRecordPtr elem =3D NULL; virDomainStatsRecordPtr *tmpret =3D NULL; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); @@ -7173,8 +7088,6 @@ remoteConnectGetAllDomainStats(virConnectPtr conn, args.stats =3D stats; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(conn, priv, 0, REMOTE_PROC_CONNECT_GET_ALL_DOMAIN_STATS, (xdrproc_t)xdr_remote_connect_get_all_domain_stats_args, (cha= r *)&args, (xdrproc_t)xdr_remote_connect_get_all_domain_stats_ret, (char= *)&ret) =3D=3D -1) { @@ -7237,7 +7150,7 @@ remoteNodeAllocPages(virConnectPtr conn, unsigned int flags) { remote_node_alloc_pages_args args; - remote_node_alloc_pages_ret ret; + remote_node_alloc_pages_ret ret =3D {0}; struct private_data *priv =3D conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -7256,7 +7169,6 @@ remoteNodeAllocPages(virConnectPtr conn, args.cellCount =3D cellCount; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(conn, priv, 0, REMOTE_PROC_NODE_ALLOC_PAGES, (xdrproc_t) xdr_remote_node_alloc_pages_args, (char *) &args, (xdrproc_t) xdr_remote_node_alloc_pages_ret, (char *) &ret) = =3D=3D -1) @@ -7275,7 +7187,7 @@ remoteDomainGetFSInfo(virDomainPtr dom, size_t i, j, len; struct private_data *priv =3D dom->conn->privateData; remote_domain_get_fsinfo_args args; - remote_domain_get_fsinfo_ret ret; + remote_domain_get_fsinfo_ret ret =3D {0}; remote_domain_fsinfo *src; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -7283,8 +7195,6 @@ remoteDomainGetFSInfo(virDomainPtr dom, =20 args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_FSINFO, (xdrproc_t)xdr_remote_domain_get_fsinfo_args, (char *)&args, (xdrproc_t)xdr_remote_domain_get_fsinfo_ret, (char *)&ret) = =3D=3D -1) @@ -7351,7 +7261,7 @@ remoteDomainInterfaceAddresses(virDomainPtr dom, =20 virDomainInterfacePtr *ifaces_ret =3D NULL; remote_domain_interface_addresses_args args; - remote_domain_interface_addresses_ret ret; + remote_domain_interface_addresses_ret ret =3D {0}; struct private_data *priv =3D dom->conn->privateData; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -7359,8 +7269,6 @@ remoteDomainInterfaceAddresses(virDomainPtr dom, args.flags =3D flags; make_nonnull_domain(&args.dom, dom); =20 - memset(&ret, 0, sizeof(ret)); - if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_INTERFACE_ADDRESSES, (xdrproc_t)xdr_remote_domain_interface_addresses_args, (char *)&args, @@ -7485,7 +7393,7 @@ remoteDomainRename(virDomainPtr dom, const char *new_= name, unsigned int flags) int rv =3D -1; struct private_data *priv =3D dom->conn->privateData; remote_domain_rename_args args; - remote_domain_rename_ret ret; + remote_domain_rename_ret ret =3D {0}; char *tmp =3D NULL; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 @@ -7495,8 +7403,6 @@ remoteDomainRename(virDomainPtr dom, const char *new_= name, unsigned int flags) args.new_name =3D new_name ? (char **)&new_name : NULL; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_RENAME, (xdrproc_t)xdr_remote_domain_rename_args, (char *)&args, (xdrproc_t)xdr_remote_domain_rename_ret, (char *)&ret) =3D=3D= -1) @@ -7521,14 +7427,12 @@ remoteStorageVolGetInfoFlags(virStorageVolPtr vol, { struct private_data *priv =3D vol->conn->privateData; remote_storage_vol_get_info_flags_args args; - remote_storage_vol_get_info_flags_ret ret; + remote_storage_vol_get_info_flags_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_storage_vol(&args.vol, vol); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(vol->conn, priv, 0, REMOTE_PROC_STORAGE_VOL_GET_INFO_FLAGS, (xdrproc_t)xdr_remote_storage_vol_get_info_flags_args, (char *)&args, @@ -7553,13 +7457,12 @@ remoteNetworkPortGetParameters(virNetworkPortPtr po= rt, int rv =3D -1; struct private_data *priv =3D port->net->conn->privateData; remote_network_port_get_parameters_args args; - remote_network_port_get_parameters_ret ret; + remote_network_port_get_parameters_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_network_port(&args.port, port); args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); if (call(port->net->conn, priv, 0, REMOTE_PROC_NETWORK_PORT_GET_PARAME= TERS, (xdrproc_t) xdr_remote_network_port_get_parameters_args, (cha= r *) &args, (xdrproc_t) xdr_remote_network_port_get_parameters_ret, (char= *) &ret) =3D=3D -1) @@ -7589,7 +7492,7 @@ remoteDomainGetGuestInfo(virDomainPtr dom, int rv =3D -1; struct private_data *priv =3D dom->conn->privateData; remote_domain_get_guest_info_args args; - remote_domain_get_guest_info_ret ret; + remote_domain_get_guest_info_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, dom); @@ -7597,8 +7500,6 @@ remoteDomainGetGuestInfo(virDomainPtr dom, args.types =3D types; args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_GUEST_INFO, (xdrproc_t)xdr_remote_domain_get_guest_info_args, (char *)&ar= gs, (xdrproc_t)xdr_remote_domain_get_guest_info_ret, (char *)&ret= ) =3D=3D -1) @@ -7629,13 +7530,12 @@ remoteDomainAuthorizedSSHKeysGet(virDomainPtr domai= n, size_t i; struct private_data *priv =3D domain->conn->privateData; remote_domain_authorized_ssh_keys_get_args args; - remote_domain_authorized_ssh_keys_get_ret ret; + remote_domain_authorized_ssh_keys_get_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.user =3D (char *) user; args.flags =3D flags; - memset(&ret, 0, sizeof(ret)); =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_AUTHORIZED_SSH_KEYS= _GET, (xdrproc_t) xdr_remote_domain_authorized_ssh_keys_get_args, (= char *)&args, @@ -7703,12 +7603,11 @@ remoteDomainGetMessages(virDomainPtr domain, size_t i; struct private_data *priv =3D domain->conn->privateData; remote_domain_get_messages_args args; - remote_domain_get_messages_ret ret; + remote_domain_get_messages_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D remoteDriverLock(priv); =20 make_nonnull_domain(&args.dom, domain); args.flags =3D flags; - memset(&ret, 0, sizeof(ret)); =20 if (call(domain->conn, priv, 0, REMOTE_PROC_DOMAIN_GET_MESSAGES, (xdrproc_t) xdr_remote_domain_get_messages_args, (char *)&arg= s, --=20 2.38.1 From nobody Sat May 18 10:08:05 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1673358878; cv=none; d=zohomail.com; s=zohoarc; b=WBq2+Sh/R3Nblp+lW80tvTSd2eCYUNoW9k6fd4By+0YTg1JEcBQ6Ef4t1NNJiAYN385JAiSwTJg08shMBP6x/a0b+btqLPHw8uBn3s+ho4wynaiSa2OaYOButZULo+Qk53HW6xfWP1akCEq4AzXoOErAdKSjFc4M+hFOjeumSQc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1673358878; h=Content-Type:Content-Transfer-Encoding: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=exixrJGJmqXuTMT78SrBU6i2WNBFnLm6Qm0vDka0Oho=; b=Lq2rtB0OifW0AQwRr6wrdOxth99UmZitVooN9BQE7ETUOAIu5XXukepbJQ6pwKxxKxCAO1leziHRTHikzvZvTZEueSEui5I4xxxFopw+KDnjCFTLw2WXltDChXHfzFQLQWOPpK/ASRDqYotleuLU6m4t9Quv7ACUaxH5Q2gX3q4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1673358878570383.7323282704216; Tue, 10 Jan 2023 05:54:38 -0800 (PST) Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-167-qevD8UkSMhSwj09TLnojbQ-1; Tue, 10 Jan 2023 08:54:33 -0500 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.rdu2.redhat.com [10.11.54.2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 213BD100F833; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (unknown [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id 0B5A34078905; Tue, 10 Jan 2023 13:54:28 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 06101194E124; Tue, 10 Jan 2023 13:54:27 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 90616194F850 for ; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 76EC940C2006; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) Received: from virtlab420.virt.lab.eng.bos.redhat.com (virtlab420.virt.lab.eng.bos.redhat.com [10.19.152.148]) by smtp.corp.redhat.com (Postfix) with ESMTP id 5C62440C2004; Tue, 10 Jan 2023 13:54:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1673358877; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=exixrJGJmqXuTMT78SrBU6i2WNBFnLm6Qm0vDka0Oho=; b=h24hCuZenZS2O/TvY24Q4L9WVgweMBUlhI8cWVWvr/ldzXnLxQ2NXL3TqBrJ74IQkIpEz/ GQHWw1W7N8KdlcOUPhG9nD28lyq9G7+FNH//B2ddcg8WNwhobOtjvWV/1AM64rQy9ehzbs QzN1lFPAu1m/HKs8XkFhWW5u76yfva0= X-MC-Unique: qevD8UkSMhSwj09TLnojbQ-1 X-Original-To: libvir-list@listman.corp.redhat.com From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: libvir-list@redhat.com Subject: [libvirt PATCH 9/9] admin: use struct zero initializer instead of memset Date: Tue, 10 Jan 2023 08:54:23 -0500 Message-Id: <20230110135423.2576907-10-berrange@redhat.com> In-Reply-To: <20230110135423.2576907-1-berrange@redhat.com> References: <20230110135423.2576907-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.1 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 3.1 on 10.11.54.2 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1673358880503100009 This is a more concise approach and guarantees there is no time window where the struct is uninitialized. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: J=C3=A1n Tomko --- src/admin/admin_remote.c | 20 +++++--------------- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/src/admin/admin_remote.c b/src/admin/admin_remote.c index 5322f087ec..3ae20ff373 100644 --- a/src/admin/admin_remote.c +++ b/src/admin/admin_remote.c @@ -232,14 +232,12 @@ remoteAdminServerGetThreadPoolParameters(virAdmServer= Ptr srv, int rv =3D -1; remoteAdminPriv *priv =3D srv->conn->privateData; admin_server_get_threadpool_parameters_args args; - admin_server_get_threadpool_parameters_ret ret; + admin_server_get_threadpool_parameters_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; make_nonnull_server(&args.srv, srv); =20 - memset(&ret, 0, sizeof(ret)); - if (call(srv->conn, 0, ADMIN_PROC_SERVER_GET_THREADPOOL_PARAMETERS, (xdrproc_t)xdr_admin_server_get_threadpool_parameters_args, (= char *) &args, (xdrproc_t)xdr_admin_server_get_threadpool_parameters_ret, (c= har *) &ret) =3D=3D -1) @@ -302,14 +300,12 @@ remoteAdminClientGetInfo(virAdmClientPtr client, int rv =3D -1; remoteAdminPriv *priv =3D client->srv->conn->privateData; admin_client_get_info_args args; - admin_client_get_info_ret ret; + admin_client_get_info_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; make_nonnull_client(&args.clnt, client); =20 - memset(&ret, 0, sizeof(ret)); - if (call(client->srv->conn, 0, ADMIN_PROC_CLIENT_GET_INFO, (xdrproc_t)xdr_admin_client_get_info_args, (char *) &args, (xdrproc_t)xdr_admin_client_get_info_ret, (char *) &ret) =3D= =3D -1) @@ -337,15 +333,13 @@ remoteAdminServerGetClientLimits(virAdmServerPtr srv, { int rv =3D -1; admin_server_get_client_limits_args args; - admin_server_get_client_limits_ret ret; + admin_server_get_client_limits_ret ret =3D {0}; remoteAdminPriv *priv =3D srv->conn->privateData; VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; make_nonnull_server(&args.srv, srv); =20 - memset(&ret, 0, sizeof(ret)); - if (call(srv->conn, 0, ADMIN_PROC_SERVER_GET_CLIENT_LIMITS, (xdrproc_t) xdr_admin_server_get_client_limits_args, (char *) &args, @@ -410,13 +404,11 @@ remoteAdminConnectGetLoggingOutputs(virAdmConnectPtr = conn, int rv =3D -1; remoteAdminPriv *priv =3D conn->privateData; admin_connect_get_logging_outputs_args args; - admin_connect_get_logging_outputs_ret ret; + admin_connect_get_logging_outputs_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(conn, 0, ADMIN_PROC_CONNECT_GET_LOGGING_OUTPUTS, @@ -442,13 +434,11 @@ remoteAdminConnectGetLoggingFilters(virAdmConnectPtr = conn, int rv =3D -1; remoteAdminPriv *priv =3D conn->privateData; admin_connect_get_logging_filters_args args; - admin_connect_get_logging_filters_ret ret; + admin_connect_get_logging_filters_ret ret =3D {0}; VIR_LOCK_GUARD lock =3D virObjectLockGuard(priv); =20 args.flags =3D flags; =20 - memset(&ret, 0, sizeof(ret)); - if (call(conn, 0, ADMIN_PROC_CONNECT_GET_LOGGING_FILTERS, --=20 2.38.1