diff options
author | Vlad Yasevich <vladislav.yasevich@hp.com> | 2008-06-19 23:08:18 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-06-19 23:08:18 (GMT) |
commit | 2e3216cd54b142ba605e87522e15f42e0c4e3996 (patch) | |
tree | da4d7a6ff4811d9f38b81c70f08b221b575e1254 /net/sctp/associola.c | |
parent | 7115e632f90952454ab6426e0d2151327162a30f (diff) | |
download | linux-2e3216cd54b142ba605e87522e15f42e0c4e3996.tar.xz |
sctp: Follow security requirement of responding with 1 packet
RFC 4960, Section 11.4. Protection of Non-SCTP-Capable Hosts
When an SCTP stack receives a packet containing multiple control or
DATA chunks and the processing of the packet requires the sending of
multiple chunks in response, the sender of the response chunk(s) MUST
NOT send more than one packet. If bundling is supported, multiple
response chunks that fit into a single packet MAY be bundled together
into one single response packet. If bundling is not supported, then
the sender MUST NOT send more than one response chunk and MUST
discard all other responses. Note that this rule does NOT apply to a
SACK chunk, since a SACK chunk is, in itself, a response to DATA and
a SACK does not require a response of more DATA.
We implement this by not servicing our outqueue until we reach the end
of the packet. This enables maximum bundling. We also identify
'response' chunks and make sure that we only send 1 packet when sending
such chunks.
Signed-off-by: Vlad Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp/associola.c')
-rw-r--r-- | net/sctp/associola.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/sctp/associola.c b/net/sctp/associola.c index 35b6a02..ff1dc5b 100644 --- a/net/sctp/associola.c +++ b/net/sctp/associola.c @@ -1025,6 +1025,7 @@ static void sctp_assoc_bh_rcv(struct work_struct *work) struct sctp_chunk *chunk; struct sock *sk; struct sctp_inq *inqueue; + struct sctp_outq *outq; int state; sctp_subtype_t subtype; int error = 0; |