]> xenbits.xensource.com Git - libvirt.git/commitdiff
rpc: use struct zero initializer instead of memset
authorDaniel P. Berrangé <berrange@redhat.com>
Thu, 22 Dec 2022 15:42:20 +0000 (10:42 -0500)
committerDaniel P. Berrangé <berrange@redhat.com>
Tue, 17 Jan 2023 17:33:54 +0000 (12:33 -0500)
This is a more concise approach and guarantees there is
no time window where the struct is uninitialized.

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

index 63d2411474a1fb822f47c28affe48bfd82bb7249..e9dc8e27d7d7aac2b167c70059a79eb6388ca776 100755 (executable)
@@ -1534,7 +1534,7 @@ elsif ($mode eq "client") {
         if ($rettype eq "void") {
             $call_ret = "NULL";
         } else {
-            push(@vars_list, "$rettype ret");
+            push(@vars_list, "$rettype ret = {0}");
 
             foreach my $ret_member (@{$call->{ret_members}}) {
                 if ($multi_ret) {
@@ -1891,11 +1891,6 @@ elsif ($mode eq "client") {
             print "\n";
         }
 
-        if ($rettype ne "void") {
-            print "\n";
-            print "    memset(&ret, 0, sizeof(ret));\n";
-        }
-
         my $callflags = "0";
         if ($structprefix eq "qemu") {
             $callflags = "REMOTE_CALL_QEMU";