Only handle one event per connection per iteration, since one event h…

…andler might close the connection before the next one runs.
dag-erling · Nov 3, 2016 · 36d1439 · 36d1439
1 parent e092017
commit 36d1439
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/src/kexkill.c b/src/kexkill.c
@@ -320,13 +320,11 @@ kexkill(struct sockaddr *sa, socklen_t salen)
 		for (i = 0; i < maxconns; ++i) {
 			if (pfd[i].fd == 0)
 				continue;
-			if (pfd[i].revents & (POLLERR|POLLHUP)) {
+			else if (pfd[i].revents & (POLLERR|POLLHUP))
 				kk_hup(&conns[i]);
-				continue;
-			}
-			if (pfd[i].revents & POLLIN)
+			else if (pfd[i].revents & POLLIN)
 				kk_input(&conns[i]);
-			if (pfd[i].revents & POLLOUT)
+			else if (pfd[i].revents & POLLOUT)
 				kk_output(&conns[i]);
 		}
 	}