Re: [libvirt] [PATCH v1 12/23] virtlockd: Work virSeclabelSpace in

Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com&gt; --- src/locking/lock_daemon.c | 44 ++++++++++++++++++++++++++ src/locking/lock_daemon.h | 8 +++++ src/locking/lock_daemon_dispatch.c | 65 ++++++++++++++++++++++++++++++++------ 3 files changed, 108 insertions(+), 9 deletions(-)

diff --git a/src/locking/lock_daemon_dispatch.c b/src/locking/lock_daemon_dispatch.c index 2d0bd81..5ad7e11 100644 --- a/src/locking/lock_daemon_dispatch.c +++ b/src/locking/lock_daemon_dispatch.c

@@ -433,21 +435,66 @@ virLockSpaceProtocolDispatchCreateLockSpace(virNetServerPtr server ATTRIBUTE_UNU

static int virLockSpaceProtocolDispatchRecallSeclabel(virNetServerPtr server ATTRIBUTE_UNUSED, - virNetServerClientPtr client ATTRIBUTE_UNUSED, + virNetServerClientPtr client, virNetMessagePtr msg ATTRIBUTE_UNUSED, - virNetMessageErrorPtr rerr ATTRIBUTE_UNUSED, - virLockSpaceProtocolRecallSeclabelArgs *args ATTRIBUTE_UNUSED, - virLockSpaceProtocolRecallSeclabelRet *ret ATTRIBUTE_UNUSED) + virNetMessageErrorPtr rerr, + virLockSpaceProtocolRecallSeclabelArgs *args, + virLockSpaceProtocolRecallSeclabelRet *ret) { - return 0; + int rv = -1; + int funcRet; + virLockDaemonClientPtr priv = + virNetServerClientGetPrivateData(client); + char *label = NULL; + char **label_p = NULL; + + virMutexLock(&priv->lock); + + memset(ret, 0, sizeof(*ret)); + + funcRet = virLockDaemonRecallSeclabel(lockDaemon, + args->path, args->model, &label); + + if (funcRet == 0 && + (VIR_ALLOC(label_p) < 0 || + VIR_STRDUP(*label_p, label) < 0))

+ goto cleanup; + + ret->label = label_p; + ret->ret = funcRet; + rv = 0;