From dfd416440670d6a14496d44996097d4f501947ca Mon Sep 17 00:00:00 2001 From: Benjamin Kaduk Date: Thu, 6 Feb 2014 17:34:21 -0500 Subject: [PATCH] aklog: those pesky assignments in conditionals This one should actually be an assignment, and the compiler warning for having two sets of parentheses is helpful. Found by clang on FreeBSD 10.0. Change-Id: Ic906cbdc50a20bcd0b91555581b60c518da2bc81 Reviewed-on: http://gerrit.openafs.org/10823 Tested-by: BuildBot Reviewed-by: Perry Ruiter Reviewed-by: Jeffrey Altman Reviewed-by: D Brashear --- src/aklog/aklog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/aklog/aklog.c b/src/aklog/aklog.c index 77d95483b..f397bf5dd 100644 --- a/src/aklog/aklog.c +++ b/src/aklog/aklog.c @@ -1856,7 +1856,7 @@ get_credv5_akimpersonate(krb5_context context, krb5_keytab_entry new[1]; int best = -1; memset(new, 0, sizeof *new); - if ((code == krb5_kt_start_seq_get(context, kt, cursor))) { + if ((code = krb5_kt_start_seq_get(context, kt, cursor))) { afs_com_err(progname, code, "while starting keytab scan"); goto cleanup; } -- 2.39.5