Now that the virAuthGet*Path API's generate all the error messages we can remove them from the callers. This means that we will no longer overwrite the error from the API. Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx> --- src/xenapi/xenapi_driver.c | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/src/xenapi/xenapi_driver.c b/src/xenapi/xenapi_driver.c index 3af5eeafcf..96cad99e4e 100644 --- a/src/xenapi/xenapi_driver.c +++ b/src/xenapi/xenapi_driver.c @@ -156,22 +156,14 @@ xenapiConnectOpen(virConnectPtr conn, virConnectAuthPtr auth, if (VIR_STRDUP(username, conn->uri->user) < 0) goto error; } else { - username = virAuthGetUsername(conn, auth, "xen", NULL, conn->uri->server); - - if (username == NULL) { - xenapiSessionErrorHandler(conn, VIR_ERR_AUTH_FAILED, - _("Username request failed")); + if (!(username = virAuthGetUsername(conn, auth, "xen", NULL, + conn->uri->server))) goto error; - } } - password = virAuthGetPassword(conn, auth, "xen", username, conn->uri->server); - - if (password == NULL) { - xenapiSessionErrorHandler(conn, VIR_ERR_AUTH_FAILED, - _("Password request failed")); + if (!(password = virAuthGetPassword(conn, auth, "xen", username, + conn->uri->server))) goto error; - } if (VIR_ALLOC(privP) < 0) goto error; -- 2.17.1 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list