ds: shutdn_tls_step clobbers {wbuf} early

Instead of prepending to {wbuf} and preserving tmpio objects
and/or other coderefs, clobber it early since we don't keep
sockets after shutting down TLS.  This ought to fix the `BUG:
ep_mod GLOB=GLOB(...): no such file or directory' (ENOENT)
errors on EPOLL_CTL_MOD when a client shuts down the connection
while serving large HTTPS responses.

diff --git a/lib/PublicInbox/DS.pm b/lib/PublicInbox/DS.pm
index ad01ffc8cd0b4546019fdb78808f5d2569578ffc..38b3635152bc2430a98b7e5d4580297a990014e9 100644 (file)
--- a/lib/PublicInbox/DS.pm
+++ b/lib/PublicInbox/DS.pm
@@ -619,7 +619,7 @@ sub shutdn_tls_step ($) {
                                $! != EAGAIN;
        my $ev = PublicInbox::TLS::epollbit() or return ds_close($self);
        epwait $sock, $ev | EPOLLONESHOT;
-       unshift @{$self->{wbuf}}, \&shutdn_tls_step; # autovivifies
+       @{$self->{wbuf}} = (\&shutdn_tls_step); # autovivifies
 }
 
 # don't bother with shutdown($sock, 2), we don't fork+exec w/o CLOEXEC