David Miller wrote: > From: Jakub Kicinski <jakub.kicin...@netronome.com> > Date: Thu, 1 Aug 2019 14:36:01 -0700 > > > Looks like we were slightly overzealous with the shutdown() > > cleanup. Even though the sock->sk_state can reach CLOSED again, > > socket->state will not got back to SS_UNCONNECTED once > > connections is ESTABLISHED. Meaning we will see EISCONN if > > we try to reconnect, and EINVAL if we try to listen. > > > > Only listen sockets can be shutdown() and reused, but since > > ESTABLISHED sockets can never be re-connected() or used for > > listen() we don't need to try to clean up the ULP state early. > > > > Fixes: 32857cf57f92 ("net/tls: fix transition through disconnect with > > close") > > Signed-off-by: Jakub Kicinski <jakub.kicin...@netronome.com> > > Applied and queued up for -stable.
Bit late but, I went back and ran some of the syzbot tests that were failing before original series and most of my ktls+bpf tests and everything seems in good shape now. There is still one issue with crypto stack that I'll look at fixing now. Thanks. Acked-by: John Fastabend <john.fastab...@gmail.com> Tested-by: John Fastabend <john.fastab...@gmail.com>