Keep track of clients that are beind killed, and don't re-add them to

the kill list.  (Bug 1100).  Update var/lyskomd.clients when the
handshake completes by a DNS response.

ChangeLog

 2003-08-22  Per Cederqvist  <ceder@ceder.dyndns.org>
 
+	Keep track of clients that are beind killed, and don't re-add them
+	to the kill list.  (Bug 1100).
+	* src/server/internal-connections.h (get_conn_by_number): Document
+	that it returns NULL if the session doesn't exist.
+	* src/server/internal-connections.c (init_connection): Set the new
+	kill_status field of Connection instead of the old kill_pending
+	field.
+	(new_client): Ditto.
+	(check_conn_exists): Dead code removed.
+	(get_conn_by_number): Don't crash if no connections exist.  It
+	isn't abnormal if the supplied connection number doesn't exist, so
+	remove code inside NDEFENSIVE_CHECKS that assumed it was.
+	Whitespace fixes.
+	* src/server/connections.h (enum kill_state): New enum.
+	(Connection): Replaced the "Bool kill_pending" field with a "enum
+	kill_stat kill_status" field.
+	* src/server/connections.c (check_kill_flg): Use the new
+	kill_status field of Connection instead of the old kill_pending
+	field.
+	(read_from_connection): Ditto.
+	(add_to_kill_list): Ditto.  Don't add the client if it is dying.
+	
+	Update var/lyskomd.clients when the handshake completes by a DNS
+	response.
+	* src/server/connections.c (dns_resolution): If the DNS resolution
+	completed the handshake, dump out the connections again to reflect
+	that fact.
+
 	Test for bug 1100.
 	* src/server/testsuite/lyskomd.0/47.exp: New file: Test client
 	disconnect while logged in and receiving async-logout.