]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUILD: evports: remove a leftover from the dead_fd cleanup
authorWilly Tarreau <w@1wt.eu>
Tue, 30 Nov 2021 08:32:21 +0000 (09:32 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 30 Nov 2021 08:34:32 +0000 (09:34 +0100)
Commit b1f29bc62 ("MINOR: activity/fd: remove the dead_fd counter") got
rid of FD_UPDT_DEAD, but evports managed to slip through the cracks and
wasn't cleaned up, thus it doesn't build anymore, as reported in github
issue #1467. We just need to remove the related lines since the situation
is already handled by the remaining conditions.

Thanks to Dominik Hassler for reporting the issue and confirming the fix.

This must be backported to 2.5 only.

src/ev_evports.c

index 73e97517cb34988a6baad32732828af757bda7c8..c3be00e7273723928f7357548c0f502eaaeee6b4 100644 (file)
@@ -250,10 +250,6 @@ static void _do_poll(struct poller *p, int exp, int wake)
                 */
                ret = fd_update_events(fd, n);
 
-               /* If the FD was already dead , skip it */
-               if (ret == FD_UPDT_DEAD)
-                       continue;
-
                /* disable polling on this instance if the FD was migrated */
                if (ret == FD_UPDT_MIGRATED) {
                        if (!HA_ATOMIC_BTS(&fdtab[fd].update_mask, tid))