----- "Daniel P. Berrange" <berrange(a)redhat.com> wrote:
@@ -635,49 +620,25 @@ secretLookupByUUIDString(virConnectPtr conn,
const char *uuid)
pptr = secretFind(driver, uuid);
if (pptr == NULL) {
+ char uuidstr[VIR_UUID_STRING_BUFLEN];
+ virUUIDFormat(uuid, uuidstr);
virSecretReportError(conn, VIR_ERR_NO_SECRET,
- _("no secret with matching id '%s'"),
uuid);
+ _("no secret with matching uuid '%s'"),
uuidstr);
goto cleanup;
}
- ret = virGetSecret(conn, (*pptr)->def->id);
+ ret = virGetSecret(conn, (*pptr)->def->uuid);
cleanup:
+ if (1) {
+ char uuidstr[VIR_UUID_STRING_BUFLEN];
+ virUUIDFormat(uuid, uuidstr);
+ VIR_ERROR("Lookup %s got %p", uuidstr, ret);
+ }
This looks like a debug print that should be removed in the final version.
Mirek