From: Ezra Peisach <epeisach@mit.edu>
Date: Tue, 21 Oct 1997 18:33:55 +0000 (+0000)
Subject: Fix to not lose entries in the chain linked client list
X-Git-Tag: krb5-1.1-beta1~984
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=a3522a0e98c4ddebfc599a8eaa1a3e81aad8a7e9;p=krb5.git

Fix to not lose entries in the chain linked client list

A pointer was not being updated resulting in the situation where
random clients would sudenly fail with a misc. rpc. error as the client
handle could not be found. The scenario required three active clients
to trigger the problem.

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@10241 dc483132-0cff-0310-8789-dd5450dbe970
---

diff --git a/src/lib/rpc/ChangeLog b/src/lib/rpc/ChangeLog
index 1a37a3e00..38be12870 100644
--- a/src/lib/rpc/ChangeLog
+++ b/src/lib/rpc/ChangeLog
@@ -1,3 +1,8 @@
+Tue Oct 21 14:31:33 1997  Ezra Peisach  <epeisach@mit.edu>
+
+	* svc_auth_gssapi.c (destroy_client): Fix to not lose entries in
+        the chain linked list.
+
 Mon Aug 18 09:48:06 1997  Ezra Peisach  <epeisach@kangaroo.mit.edu>
 
 	* get_myaddress.c: Include string.h for memcpy prototype.
diff --git a/src/lib/rpc/svc_auth_gssapi.c b/src/lib/rpc/svc_auth_gssapi.c
index 20bf95a74..d69a43ecb 100644
--- a/src/lib/rpc/svc_auth_gssapi.c
+++ b/src/lib/rpc/svc_auth_gssapi.c
@@ -5,6 +5,14 @@
  * $Source$
  * 
  * $Log$
+ * Revision 1.43  1997/10/21 18:33:55  epeisach
+ * Fix to not lose entries in the chain linked client list.
+ *
+ * A pointer was not being updated resulting in the situation where
+ * random clients would sudenly fail with a misc. rpc. error as the client
+ * handle could not be found. The scenario required three active clients
+ * to trigger the problem.
+ *
  * Revision 1.42  1996/12/04 17:47:18  bjaspan
  * 	* Various changes to allow channel bindings to work with both UDP
  *  	and TCP cleanly [krb5-libs/180]:
@@ -1015,8 +1023,10 @@ static void destroy_client(client_data)
 		    c2->next = c->next;
 		    free(c);
 		    goto done;
-	       } else
+	       } else {
+		    c2 = c;
 		    c = c->next;
+	       }
 	  }
 	  PRINTF(("destroy_client: client_handle delete failed\n"));
 	  abort();