diff mbox series

[pseudo,18/34] pseudo_server: Lower highest_client when the highest client closes

Message ID 1794370862-30908-19-git-send-email-mark.hatle@kernel.crashing.org
State New
Headers show
Series Various fixes | expand

Commit Message

Mark Hatle Nov. 11, 2026, 4:20 a.m. UTC 
From: Mark Hatle <mark.hatle@amd.com>

close_client() is meant to move highest_client down past closed slots
when the highest client goes away, but the loop condition was inverted:
it stopped at the first closed slot, which is always the one just
closed, so highest_client never went down.

AI-Generated: Uses GitHub Copilot
Signed-off-by: Mark Hatle <mark.hatle@amd.com>
---
 pseudo_server.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/pseudo_server.c b/pseudo_server.c
index c54c0d2..edcb23e 100644
--- a/pseudo_server.c
+++ b/pseudo_server.c
@@ -441,7 +441,7 @@  close_client(int client) {
 	clients[client].program = NULL;
 	--active_clients;
 	if (client == highest_client)
-		while (clients[highest_client].fd != -1 && highest_client > 0)
+		while (clients[highest_client].fd == -1 && highest_client > 0)
 			--highest_client;
 }