mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-01-03 11:35:19 +00:00
util: auth: Introduce virAuthAskCredential
The helper uses the user-provided auth callbacks to ask the user. The helper encapsulates the steps we do to query the user in few places into a common helper which can be then used further. Signed-off-by: Peter Krempa <pkrempa@redhat.com> Reviewed-by: Jonathon Jongsma <jjongsma@redhat.com>
This commit is contained in:
parent
7fb0c7418e
commit
95f4879e97
@ -1850,6 +1850,8 @@ virAuditSend;
|
|||||||
|
|
||||||
|
|
||||||
# util/virauth.h
|
# util/virauth.h
|
||||||
|
virAuthAskCredential;
|
||||||
|
virAuthConnectCredentialFree;
|
||||||
virAuthGetConfigFilePath;
|
virAuthGetConfigFilePath;
|
||||||
virAuthGetConfigFilePathURI;
|
virAuthGetConfigFilePathURI;
|
||||||
virAuthGetPassword;
|
virAuthGetPassword;
|
||||||
|
@ -31,6 +31,7 @@
|
|||||||
#include "virerror.h"
|
#include "virerror.h"
|
||||||
#include "configmake.h"
|
#include "configmake.h"
|
||||||
#include "virauthconfig.h"
|
#include "virauthconfig.h"
|
||||||
|
#include "virsecureerase.h"
|
||||||
|
|
||||||
#define VIR_FROM_THIS VIR_FROM_AUTH
|
#define VIR_FROM_THIS VIR_FROM_AUTH
|
||||||
|
|
||||||
@ -283,3 +284,68 @@ virAuthGetPassword(virConnectPtr conn,
|
|||||||
|
|
||||||
return virAuthGetPasswordPath(path, auth, servicename, username, hostname);
|
return virAuthGetPasswordPath(path, auth, servicename, username, hostname);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
void
|
||||||
|
virAuthConnectCredentialFree(virConnectCredential *cred)
|
||||||
|
{
|
||||||
|
if (cred->result) {
|
||||||
|
virSecureErase(cred->result, cred->resultlen);
|
||||||
|
g_free(cred->result);
|
||||||
|
}
|
||||||
|
g_free(cred);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
/**
|
||||||
|
* virAuthAskCredential:
|
||||||
|
* @auth: authentication callback data
|
||||||
|
* @prompt: question string to ask the user
|
||||||
|
* @echo: false if user's reply should be considered sensitive and not echoed
|
||||||
|
*
|
||||||
|
* Invoke the authentication callback for the connection @auth and ask the user
|
||||||
|
* the question in @prompt. If @echo is false user's reply should be collected
|
||||||
|
* as sensitive (user's input not printed on screen).
|
||||||
|
*/
|
||||||
|
virConnectCredential *
|
||||||
|
virAuthAskCredential(virConnectAuthPtr auth,
|
||||||
|
const char *prompt,
|
||||||
|
bool echo)
|
||||||
|
{
|
||||||
|
g_autoptr(virConnectCredential) ret = g_new0(virConnectCredential, 1);
|
||||||
|
size_t i;
|
||||||
|
|
||||||
|
ret->type = -1;
|
||||||
|
|
||||||
|
for (i = 0; i < auth->ncredtype; ++i) {
|
||||||
|
int type = auth->credtype[i];
|
||||||
|
if (echo) {
|
||||||
|
if (type == VIR_CRED_ECHOPROMPT) {
|
||||||
|
ret->type = type;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
if (type == VIR_CRED_PASSPHRASE ||
|
||||||
|
type == VIR_CRED_NOECHOPROMPT) {
|
||||||
|
ret->type = type;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (ret->type == -1) {
|
||||||
|
virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
|
||||||
|
_("no suitable callback authentication callback was found"));
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
ret->prompt = prompt;
|
||||||
|
|
||||||
|
if (auth->cb(ret, 1, auth->cbdata) < 0) {
|
||||||
|
virReportError(VIR_ERR_OPERATION_FAILED, "%s",
|
||||||
|
_("failed to retrieve user response for authentication callback"));
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
return g_steal_pointer(&ret);
|
||||||
|
}
|
||||||
|
@ -52,3 +52,10 @@ char * virAuthGetPasswordPath(const char *path,
|
|||||||
const char *servicename,
|
const char *servicename,
|
||||||
const char *username,
|
const char *username,
|
||||||
const char *hostname);
|
const char *hostname);
|
||||||
|
|
||||||
|
virConnectCredential *virAuthAskCredential(virConnectAuthPtr auth,
|
||||||
|
const char *prompt,
|
||||||
|
bool echo);
|
||||||
|
|
||||||
|
void virAuthConnectCredentialFree(virConnectCredential *cred);
|
||||||
|
G_DEFINE_AUTOPTR_CLEANUP_FUNC(virConnectCredential, virAuthConnectCredentialFree);
|
||||||
|
Loading…
Reference in New Issue
Block a user