LU-4307 kerberos: compile errors with gss/kerberos support

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 <thomas@stibor.net>
Change-Id: I20b0b0ffa3ea34ce0a954146ccc762b7c17c00de
Reviewed-on: http://review.whamcloud.com/8393
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andrew Korty <ajk@iu.edu>
Reviewed-by: Nathaniel Clark <nathaniel.l.clark@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>

diff --git a/lustre/utils/gss/context_lucid.c b/lustre/utils/gss/context_lucid.c
index 7eba6b6..e95d11e 100644 (file)
--- 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 (file)
--- 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;
 }
 
 /*