From b9979e1a23ce28b20fe95d5e6042b9b15506c993 Mon Sep 17 00:00:00 2001 From: Michael Meffie Date: Fri, 9 May 2014 11:43:35 -0400 Subject: [PATCH] aklog: double get_credv5 in retry logic Fix a bug where get_credv5() is called twice, even if the first one succeeded, in the aklog retry logic. (This bug uncovers another bug where get_credv5_akimpersonate crashes in the krb5 libs when get_credv5 is called back to back.) Change-Id: Ie6cac3b4522946c87c30ad3cd6939738234800bf Reviewed-on: http://gerrit.openafs.org/11133 Reviewed-by: Benjamin Kaduk Tested-by: BuildBot Reviewed-by: Jeffrey Altman --- src/aklog/aklog.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/aklog/aklog.c b/src/aklog/aklog.c index f397bf5dd..eaead4478 100644 --- a/src/aklog/aklog.c +++ b/src/aklog/aklog.c @@ -569,9 +569,9 @@ rxkad_get_ticket(krb5_context context, char *realm, } afs_dprintf("We've deduced that we need to authenticate" " to realm %s.\n", realm_of_cell); + status = get_credv5(context, AFSKEY, cell->name, + realm_of_cell, v5cred); } - status = get_credv5(context, AFSKEY, cell->name, - realm_of_cell, v5cred); } /* If the realm and cell name match, then try without an -- 2.39.5