lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 29 Jun 2011 11:48:14 -0400 From: Thomas Graf <tgraf@...radead.org> To: Vladislav Yasevich <vladislav.yasevich@...com> Cc: netdev@...r.kernel.org, davem@...emloft.net, Wei Yongjun <yjwei@...fujitsu.com>, Sridhar Samudrala <sri@...ibm.com>, linux-sctp@...r.kernel.org Subject: Re: [PATCH] sctp: Enforce maximum retransmissions during shutdown On Wed, Jun 29, 2011 at 10:58:41AM -0400, Vladislav Yasevich wrote: > But what you are proposing violates the protocol. Zero-window probes do > not count against max retransmissions, even when you are in shutdown pending > state. > > You'll come out of this one of 2 ways: > 1) receiver wakes up and processes data. This will allow for graceful termination. > 2) receiver dies. Since receive window is full, we have data queued, and this will > trigger an ABORT to be sent to the sender. If by die you mean kill the process then this is exactly what I do to trigger the issue. I simulatenously kill both processes. Not sure what you mean by trigger an ABORT but I don't see that happen. I also don't see the rwnd reopen after the socket is closed on the receiver side but that's a separate issue. > What you patch is doing is taking a perfectly valid scenario and putting a time limit > on it in violation of the spec. We also violate the spec by not doing so. The spec says that the number of SHUTDOWN retransmissions has to be limited by Max.Retrans which we also can't enforce because of the above. The scenario is closed sockets on both sides, endpoints on both sides gone already and retransmissions + heartbeat requests forever. Any alternative suggestion how to fix this? -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists