From nobody Tue Feb 10 06:05:31 2026 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 --- 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