From b8ab46699c4ca7f6bf75205f1a9b7ec123ba9aa2 Mon Sep 17 00:00:00 2001 From: Thomas Stibor Date: Mon, 25 Nov 2013 16:41:21 +0100 Subject: [PATCH] LU-4307 kerberos: compile errors with gss/kerberos support MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Configure Lustre with parameters --with-krb5 --enable-gss causes compile errors due to missing '*' and unused variable. The errors are: taking address of expression of type ‘void’ [-Werror] and variable ‘res’ set but not used [-Werror=unused-but-set-variable] Signed-off-by: Thomas Stibor Change-Id: I20b0b0ffa3ea34ce0a954146ccc762b7c17c00de Reviewed-on: http://review.whamcloud.com/8393 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Andrew Korty Reviewed-by: Nathaniel Clark Reviewed-by: Oleg Drokin --- lustre/utils/gss/context_lucid.c | 8 ++++---- lustre/utils/gss/lgss_keyring.c | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lustre/utils/gss/context_lucid.c b/lustre/utils/gss/context_lucid.c index 7eba6b6..e95d11e 100644 --- a/lustre/utils/gss/context_lucid.c +++ b/lustre/utils/gss/context_lucid.c @@ -204,10 +204,10 @@ enum seal_alg { * We don't have "legal" access to these MIT-only * structures located in libk5crypto */ -extern void krb5int_enc_arcfour; -extern void krb5int_enc_des3; -extern void krb5int_enc_aes128; -extern void krb5int_enc_aes256; +extern void *krb5int_enc_arcfour; +extern void *krb5int_enc_des3; +extern void *krb5int_enc_aes128; +extern void *krb5int_enc_aes256; static void key_lucid_to_krb5(const gss_krb5_lucid_key_t *lin, krb5_keyblock *kout) diff --git a/lustre/utils/gss/lgss_keyring.c b/lustre/utils/gss/lgss_keyring.c index 04a80b7..7e28ce2 100644 --- a/lustre/utils/gss/lgss_keyring.c +++ b/lustre/utils/gss/lgss_keyring.c @@ -195,9 +195,9 @@ int do_nego_rpc(struct lgss_nego_data *lnd, memcpy(gr->gr_token.value, p, gr->gr_token.length); p += (((gr->gr_token.length + 3) & ~3) / 4); - logmsg(LL_DEBUG, "do_nego_rpc: receive handle len %d, token len %d\n", - gr->gr_ctx.length, gr->gr_token.length); - return 0; + logmsg(LL_DEBUG, "do_nego_rpc: receive handle len %d, token len %d, " \ + "res %d\n", gr->gr_ctx.length, gr->gr_token.length, res); + return 0; } /* -- 1.8.3.1