[libvirt RFCv5 11/27] gendispatch: add DomainRestoreParams as requiring conn argument

Claudio Fontana cfontana at suse.de
Sat Apr 30 14:15:46 UTC 2022


in preparation for the addition of DomainRestoreParams,
add it to the list of methods requiring a conn first argument.

Signed-off-by: Claudio Fontana <cfontana at suse.de>
---
 src/rpc/gendispatch.pl | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/rpc/gendispatch.pl b/src/rpc/gendispatch.pl
index 9f5bf0e316..a64ff3e73f 100755
--- a/src/rpc/gendispatch.pl
+++ b/src/rpc/gendispatch.pl
@@ -637,7 +637,10 @@ elsif ($mode eq "server") {
                 } elsif ($args_member =~ m/^remote_typed_param (\S+)<(\S+)>;/) {
                     push(@vars_list, "virTypedParameterPtr $1 = NULL");
                     push(@vars_list, "int n$1 = 0");
-                    if ($call->{ProcName} eq "NodeSetMemoryParameters") {
+
+                    # NB: if your new API starts with remote_typed_params, enter it here if you need
+                    # the conn arg to be passed first!
+                    if ($call->{ProcName} eq "NodeSetMemoryParameters" || $call->{ProcName} eq "DomainRestoreParams") {
                         push(@args_list, $conn_var);
                     }
                     push(@args_list, "$1");
-- 
2.34.1



More information about the libvir-list mailing list