On 09/05/12 08:28, Osier Yang wrote:
The RPC generator doesn't support returning list of object yet, this patch
do the work manually.

   * daemon/remote.c:
     Implemente the server side handler remoteDispatchConnectListAllSecrets.

   * src/remote/remote_driver.c:
     Add remote driver handler remoteConnectListAllSecrets.

   * src/remote/remote_protocol.x:
     New RPC procedure REMOTE_PROC_CONNECT_LIST_ALL_SECRETS and
     structs to represent the args and ret for it.

   * src/remote_protocol-structs: Likewise.
---
  daemon/remote.c              |   54 +++++++++++++++++++++++++++++++++++
  src/remote/remote_driver.c   |   64 ++++++++++++++++++++++++++++++++++++++++++
  src/remote/remote_protocol.x |   13 ++++++++-
  src/remote_protocol-structs  |   12 ++++++++
  4 files changed, 142 insertions(+), 1 deletions(-)


...

diff --git a/src/remote_protocol-structs b/src/remote_protocol-structs
index 0dde9fc..afd3608 100644
--- a/src/remote_protocol-structs
+++ b/src/remote_protocol-structs
@@ -2078,6 +2078,17 @@ struct remote_connect_list_all_nwfilters_ret {
          } filters;
          u_int                      ret;
  };
+struct remote_list_all_secrets_args {
+        int                        need_results;
+        u_int                      flags;
+};
+struct remote_list_all_secrets_ret {
+        struct {
+                u_int              secrets_len;
+                remote_nonnull_network * secrets_val;

This should be remote_nonnull_secret.

+        } secrets;
+        u_int                      ret;
+};
  enum remote_procedure {
          REMOTE_PROC_OPEN = 1,
          REMOTE_PROC_CLOSE = 2,
@@ -2365,4 +2376,5 @@ enum remote_procedure {
          REMOTE_PROC_CONNECT_LIST_ALL_INTERFACES = 284,
          REMOTE_PROC_CONNECT_LIST_ALL_NODE_DEVICES = 285,
          REMOTE_PROC_CONNECT_LIST_ALL_NWFILTERS = 286,
+        REMOTE_PROC_CONNECT_LIST_ALL_SECRETS = 287,
  };


Otherwise looks good.

Peter

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to