diff options
| author | Ondrej Kos <okos@redhat.com> | 2013-06-24 16:55:02 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-07-10 17:12:23 +0200 |
| commit | 6c18f3287e4d4549ce2ce833dd8904d37c00c557 (patch) | |
| tree | 382cd80b8705afb85b1cf73d9b7669125dca5fc9 | |
| parent | aed8c0cae4f53926b4c34bb41e47103367c2815b (diff) | |
| download | sssd-6c18f3287e4d4549ce2ce833dd8904d37c00c557.tar.gz sssd-6c18f3287e4d4549ce2ce833dd8904d37c00c557.tar.xz sssd-6c18f3287e4d4549ce2ce833dd8904d37c00c557.zip | |
KRB5_CHILD: Fix handling of get_password return code
The switch statement was dead code due to missing case/default.
| -rw-r--r-- | src/providers/krb5/krb5_child.c | 14 |
1 files 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); |