Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The Forward-TSN-supported parameter is not listed at the unrecognized parameters #2

Open
TheAomx opened this issue Dec 12, 2016 · 4 comments
Assignees
Labels

Comments

@TheAomx
Copy link

TheAomx commented Dec 12, 2016

When an INIT_ACK-Chunk like

INIT_ACK[flgs=0, tag=2, a_rwnd=1500, os=16, is=16, tsn=1, STATE_COOKIE[len=4, val=...], FORWARD_TSN_SUPPORTED[]]

is injected and PR-SCTP-Support is disabled at FreeBSD, it is not listed at the unrecognized parameters at the COOKIE_ECHO-Chunk. See the following test-cases for more details:

@stewrtrs
Copy link
Member

stewrtrs commented Dec 12, 2016 via email

@TheAomx
Copy link
Author

TheAomx commented Dec 12, 2016

Section 3.3.2. of RFC 3758:

When a receiver of an INIT-ACK detects a Forward-TSN-Supported
parameter and it does not support the Forward-TSN chunk type, the
receiver MUST follow the rules defined in Section 3.3.3 of RFC 2960
[2].

The Linux Implementation does follow this rule if the extension has only been disabled with sysctl...

@tuexen
Copy link
Member

tuexen commented Dec 12, 2016

Hi Randy,

I reacted exactly as you.

But we/someone wrote in RFC3758:

Note that if the endpoint chooses NOT to include the parameter, then
at no time during the life of the association can it send or process
a FORWARD TSN.  It MUST instead act as if it does NOT support the
FORWARD TSN chunk, returning an ERROR to the peer upon receipt of any
FORWARD TSN.

in addition to

When a receiver of an INIT detects a Forward-TSN-Supported parameter
and does not support the Forward-TSN chunk type, the receiver MUST
follow the rules defined in Section 3.3.3 of RFC 2960 [2].

This text is not as I would expect, but it is pretty clear... What do you think?

@stewrtrs
Copy link
Member

stewrtrs commented Dec 12, 2016 via email

@tuexen tuexen self-assigned this Dec 12, 2016
@tuexen tuexen added the bug label Dec 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants