From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Sat, 31 Mar 2007 13:52:17 +0000 (+0000)
Subject: windows-nim-plugin-20070331
X-Git-Tag: BP-openafs-windows-kdfs-ifs~747
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=20890b4e2edb6d1803148493e0f83d44c4e8a8d6;p=packages%2Fo%2Fopenafs.git

windows-nim-plugin-20070331


correct a NULL pointer de-reference introduced in
windows-nim-plugin-20070328
---

diff --git a/src/WINNT/netidmgr_plugin/afsfuncs.c b/src/WINNT/netidmgr_plugin/afsfuncs.c
index 2b4ea4de0..9a6e7a17e 100644
--- a/src/WINNT/netidmgr_plugin/afsfuncs.c
+++ b/src/WINNT/netidmgr_plugin/afsfuncs.c
@@ -880,7 +880,7 @@ afs_klog(khm_handle identity,
 	* and try again.  Perhaps there are two service tickets for the
 	* same service in the ccache.
 	*/
-	if (k5creds->times.endtime < time(NULL)) {
+	if (r == 0 && k5creds && k5creds->times.endtime < time(NULL)) {
 	    pkrb5_cc_remove_cred(context, k5cc, 0, k5creds);
 	    pkrb5_free_creds(context, k5creds);
 	    k5creds = NULL;