Fix for possible NULL pointer passing to LOG if KRB5_KTNAME is not set in the process...

diff --git a/etc/uams/uams_gss.c b/etc/uams/uams_gss.c
index fa5283ec618afe958645fe661282292a7f78e606..2a1e85e2b5b33c01d7696279d37f18c9cff6abcb 100644 (file)
--- a/etc/uams/uams_gss.c
+++ b/etc/uams/uams_gss.c
@@ -1,5 +1,5 @@
 /*
- * $Id: uams_gss.c,v 1.6 2009-09-28 12:16:49 franklahm Exp $
+ * $Id: uams_gss.c,v 1.7 2009-09-28 13:19:48 franklahm Exp $
  *
  * Copyright (c) 1990,1993 Regents of The University of Michigan.
  * Copyright (c) 1999 Adrian Sun (asun@u.washington.edu)
@@ -319,10 +319,17 @@ static int wrap_sessionkey(gss_ctx_id_t context, struct session_info *sinfo)
 static int acquire_credentials (gss_name_t *server_name, gss_cred_id_t *server_creds)
 {
     OM_uint32 major_status = 0, minor_status = 0;
-
-    LOG(log_debug, logtype_uams,
-        "acquire credentials: acquiring credentials (uid = %d, keytab = %s)",
-        (int)geteuid(), getenv( "KRB5_KTNAME") );
+    char *envp;
+
+    if (envp = getenv("KRB5_KTNAME"))
+        LOG(log_debug, logtype_uams,
+            "acquire credentials: acquiring credentials (uid = %d, keytab = %s)",
+            (int)geteuid(), envp);
+    else
+        LOG(log_debug, logtype_uams,
+            "acquire credentials: acquiring credentials (uid = %d) - $KRB5_KTNAME not found in env",
+            (int)geteuid());
+        
     /*
      * Acquire credentials usable for accepting context negotiations.
      * Credentials are for server_name, have an indefinite lifetime,