client: don't check FD_ISSET(client->fd) on expired client

client->fd becomes -1 when the client expires. Don't use FD_ISSET() with this expired client; doing so would cause a crash due to SIGBUS.

client: don't check FD_ISSET(client->fd) on expired client
6146d4f5 · Max Kellermann · 77b16716 · 6146d4f5
Commit 6146d4f5 authored Sep 06, 2008 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

client.c src/client.c +2 -1

No files found.
--- a/src/client.c
+++ b/src/client.c
@@ -540,7 +540,8 @@ int client_manager_io(void)
 			}
 			client->lastTime = time(NULL);
 		}
-		if (FD_ISSET(client->fd, &wfds)) {
+		if (!client_is_expired(client) &&
+		    FD_ISSET(client->fd, &wfds)) {
 			client_write_deferred(client);
 			client->lastTime = time(NULL);
 		}