From 1b50f2d1021f56b46de0bf280897886f02d3adb8 Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Tue, 2 Mar 2021 14:18:05 +0100 Subject: [PATCH] virNetLibsshAuthenticatePrivkeyCb: Use g_autofree for 'actual_prompt' MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit So that the 'error' label can be removed. Signed-off-by: Peter Krempa Reviewed-by: Ján Tomko --- src/rpc/virnetlibsshsession.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/src/rpc/virnetlibsshsession.c b/src/rpc/virnetlibsshsession.c index 8814487557..0fc1f66706 100644 --- a/src/rpc/virnetlibsshsession.c +++ b/src/rpc/virnetlibsshsession.c @@ -406,7 +406,7 @@ virNetLibsshAuthenticatePrivkeyCb(const char *prompt, virNetLibsshSessionPtr sess = userdata; virConnectCredential retr_passphrase; int cred_type; - char *actual_prompt = NULL; + g_autofree char *actual_prompt = NULL; int p; /* request user's key password */ @@ -421,7 +421,7 @@ virNetLibsshAuthenticatePrivkeyCb(const char *prompt, if (cred_type == -1) { virReportError(VIR_ERR_LIBSSH, "%s", _("no suitable callback for input of key passphrase")); - goto error; + return -1; } actual_prompt = g_strndup(prompt, virLengthForPromptString(prompt)); @@ -434,7 +434,7 @@ virNetLibsshAuthenticatePrivkeyCb(const char *prompt, virReportError(VIR_ERR_LIBSSH, "%s", _("failed to retrieve private key passphrase: " "callback has failed")); - goto error; + return -1; } p = virStrncpy(buf, retr_passphrase.result, @@ -444,16 +444,10 @@ virNetLibsshAuthenticatePrivkeyCb(const char *prompt, if (p < 0) { virReportError(VIR_ERR_LIBSSH, "%s", _("passphrase is too long for the buffer")); - goto error; + return -1; } - VIR_FREE(actual_prompt); - return 0; - - error: - VIR_FREE(actual_prompt); - return -1; } static int