diff options
author | Günther Deschner <gd@samba.org> | 2007-06-22 14:50:15 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:23:33 -0500 |
commit | 0b9acc8610ae2ba9c42168e9ceb2e9ea8bc2f5bd (patch) | |
tree | eda2bd5018939da2d593addc2fab8e6dd81fc103 | |
parent | 1e4a7af99303fb17ebca499ff7e0a017a2017754 (diff) | |
download | samba-0b9acc8610ae2ba9c42168e9ceb2e9ea8bc2f5bd.tar.gz samba-0b9acc8610ae2ba9c42168e9ceb2e9ea8bc2f5bd.tar.xz samba-0b9acc8610ae2ba9c42168e9ceb2e9ea8bc2f5bd.zip |
r23587: Cleanup redundant code in the krb5 renew function.
Guenther
-rw-r--r-- | source/libsmb/clikrb5.c | 39 |
1 files changed, 16 insertions, 23 deletions
diff --git a/source/libsmb/clikrb5.c b/source/libsmb/clikrb5.c index 0180bef47ae..be6fb1bda8f 100644 --- a/source/libsmb/clikrb5.c +++ b/source/libsmb/clikrb5.c @@ -1159,22 +1159,22 @@ out: goto done; } + if (client_string) { + ret = smb_krb5_parse_name(context, client_string, &client); + if (ret) { + goto done; + } + } else { + ret = krb5_cc_get_principal(context, ccache, &client); + if (ret) { + goto done; + } + } + #ifdef HAVE_KRB5_GET_RENEWED_CREDS /* MIT */ { krb5_creds creds; - - if (client_string) { - ret = smb_krb5_parse_name(context, client_string, &client); - if (ret) { - goto done; - } - } else { - ret = krb5_cc_get_principal(context, ccache, &client); - if (ret) { - goto done; - } - } - + ret = krb5_get_renewed_creds(context, &creds, client, ccache, CONST_DISCARD(char *, service_string)); if (ret) { DEBUG(10,("smb_krb5_renew_ticket: krb5_get_kdc_cred failed: %s\n", error_message(ret))); @@ -1204,16 +1204,9 @@ out: memset(&creds_in, 0, sizeof(creds_in)); - if (client_string) { - ret = smb_krb5_parse_name(context, client_string, &creds_in.client); - if (ret) { - goto done; - } - } else { - ret = krb5_cc_get_principal(context, ccache, &creds_in.client); - if (ret) { - goto done; - } + ret = krb5_copy_principal(context, client, &creds_in.client); + if (ret) { + goto done; } if (service_string) { |