From 0c0f91311fd2a947992914d8bca644cd1eb4298b Mon Sep 17 00:00:00 2001 From: Ondrej Kos Date: Mon, 24 Jun 2013 16:55:02 +0200 Subject: KRB5_CHILD: Fix handling of get_password return code The switch statement was dead code due to missing case/default. --- src/providers/krb5/krb5_child.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/src/providers/krb5/krb5_child.c b/src/providers/krb5/krb5_child.c index 9480d2b3a..4d12b90a2 100644 --- a/src/providers/krb5/krb5_child.c +++ b/src/providers/krb5/krb5_child.c @@ -1451,12 +1451,18 @@ static errno_t tgt_req_child(struct krb5_req *kr) ret = sss_authtok_get_password(kr->pd->authtok, &password, NULL); switch (ret) { - if (ret == EACCES) { + case EOK: + break; + + case EACCES: DEBUG(SSSDBG_OP_FAILURE, ("Invalid authtok type\n")); return ERR_INVALID_CRED_TYPE; - } - DEBUG(SSSDBG_OP_FAILURE, ("No credentials available\n")); - return ERR_NO_CREDS; + break; + + default: + DEBUG(SSSDBG_OP_FAILURE, ("No credentials available\n")); + return ERR_NO_CREDS; + break; } kerr = get_and_save_tgt(kr, password); -- cgit