]> xenbits.xensource.com Git - libvirt.git/commitdiff
remote: remove redundant initialization of args variable
authorDaniel P. Berrangé <berrange@redhat.com>
Thu, 22 Dec 2022 17:20:32 +0000 (12:20 -0500)
committerDaniel P. Berrangé <berrange@redhat.com>
Tue, 17 Jan 2023 17:33:54 +0000 (12:33 -0500)
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.

Reviewed-by: Ján Tomko <jtomko@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
src/remote/remote_driver.c

index 43a714c79a3b80355685804203d6660b488f29c6..deeabdb24f80ad1cb3f4ca34ddf0a39fb09d4ee5 100644 (file)
@@ -6000,7 +6000,6 @@ remoteDomainMigrateBegin3(virDomainPtr domain,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     make_nonnull_domain(&args.dom, domain);
@@ -6056,7 +6055,6 @@ remoteDomainMigratePrepare3(virConnectPtr dconn,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     args.cookie_in.cookie_in_val = (char *)cookiein;
@@ -6125,7 +6123,6 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dconn,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     if (!(netst = virNetClientStreamNew(priv->remoteProgram,
@@ -6200,7 +6197,6 @@ remoteDomainMigratePerform3(virDomainPtr dom,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     make_nonnull_domain(&args.dom, dom);
@@ -6260,7 +6256,6 @@ remoteDomainMigrateFinish3(virConnectPtr dconn,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     args.cookie_in.cookie_in_val = (char *)cookiein;
@@ -6314,8 +6309,6 @@ remoteDomainMigrateConfirm3(virDomainPtr domain,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
-
     make_nonnull_domain(&args.dom, domain);
     args.cookie_in.cookie_in_len = cookieinlen;
     args.cookie_in.cookie_in_val = (char *) cookiein;
@@ -6908,7 +6901,6 @@ remoteDomainMigrateBegin3Params(virDomainPtr domain,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     make_nonnull_domain(&args.dom, domain);
@@ -6971,7 +6963,6 @@ remoteDomainMigratePrepare3Params(virConnectPtr dconn,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     if (virTypedParamsSerialize(params, nparams,
@@ -7047,7 +7038,6 @@ remoteDomainMigratePrepareTunnel3Params(virConnectPtr dconn,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     args.cookie_in.cookie_in_val = (char *)cookiein;
@@ -7129,7 +7119,6 @@ remoteDomainMigratePerform3Params(virDomainPtr dom,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     make_nonnull_domain(&args.dom, dom);
@@ -7195,7 +7184,6 @@ remoteDomainMigrateFinish3Params(virConnectPtr dconn,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
     memset(&ret, 0, sizeof(ret));
 
     args.cookie_in.cookie_in_val = (char *)cookiein;
@@ -7261,8 +7249,6 @@ remoteDomainMigrateConfirm3Params(virDomainPtr domain,
 
     remoteDriverLock(priv);
 
-    memset(&args, 0, sizeof(args));
-
     make_nonnull_domain(&args.dom, domain);
     args.cookie_in.cookie_in_len = cookieinlen;
     args.cookie_in.cookie_in_val = (char *) cookiein;
@@ -7563,8 +7549,6 @@ remoteConnectGetAllDomainStats(virConnectPtr conn,
     virDomainStatsRecordPtr elem = NULL;
     virDomainStatsRecordPtr *tmpret = NULL;
 
-    memset(&args, 0, sizeof(args));
-
     if (ndoms) {
         args.doms.doms_val = g_new0(remote_nonnull_domain, ndoms);