[tsvwg] 回复： Some unclear points about rfc4960 (SCTP).

[一念之后★だ <[e-mail]>]

HI,
I am sorry for late reply. I combined the reply for several emails: 



Which version of the Linux kernel are you using (what is the output of uname -a)?
 
Zhouming: I test ubuntu16.04 LTS and ubuntu20.04 LTS, they both have these problems.

 uname -a



Linux ubuntu 4.4.0-142-generic #168-Ubuntu SMP Wed Jan 16 21:00:45 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
 
# uname -a
 
Linux ubuntu 5.11.0-27-generic #29~20.04.1-Ubuntu SMP Wed Aug 11 15:58:17 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

NCAT version:   Ncat: Version 7.80 ( https://nmap.org/ncat )




---------------

I agree that the your attached .pcap file indicates that the implementation used has the bug you are describing. I used packetdrill (from https://github.com/nplab/packetdrill) with the  attached script on a 5.11 kernel, and the attached script passes, which means that that stack does not have the problem. I'm also attaching the corresponding .pcapng file.

So it seems that this bug was fixed recently in Linux, or am I missing something?

 Zhouming:  I checked the source code of 5.11 kernel, and find nothing fix about the problems we discussed. I guess maybe you missed something, I have no idea if packetdrill is suitable for this test.




---------------

> This has already been clarified in draft-ietf-tsvwg-rfc4960-bis-13.
>
> It states now:
&gt; <t&gt;If the value of the Initiate Tag in a received INIT chunk is found to be 0,
&gt; the receiver MUST treat it as an error and silently discard the packet.</t&gt;
&gt;
&gt; For OS = 0 or MIS = 0, the INIT chunk MUST be dropped and an ABORT chunk
&gt; SHOULD be sent. So the next revision will contain:
&gt;
&gt; <t&gt;A receiver of an INIT chunk with the OS value set to 0 MUST discard the
&gt; packet and SHOULD send a packet in response containing an ABORT chunk and using
&gt; the Initiate Tag as the Verification Tag.
&gt; Any existing association MUST NOT be affected.</t&gt;
&gt;
&gt; <t&gt;A receiver of an INIT chunk with the MIS value set to 0 MUST discard the
&gt; packet and SHOULD send a packet in response containing an ABORT chunk and using
&gt; the Initiate Tag as the Verification Tag.
&gt; Any existing association MUST NOT be affected.</t&gt;
&gt;
&gt; I guess this makes the description for the INIT chunk crystal clear.

Zhouming:&nbsp; situation(2) is missing in&nbsp;draft-ietf-tsvwg-rfc4960-bis-13 or it no need to be&nbsp;clarified&nbsp;&nbsp;??&nbsp; &nbsp; I have test this, an init msg with&nbsp;mandatory parameters not fully carried can also&nbsp;trigger an Abort msg to release the association.

---

As can be seen from the above description, when an init message or init ACK message is received,


(1)If the init tag in the message is 0, the association must be released


(2)If the mandatory parameters in the message are not fully carried, the association must be released




(3)When the OS value or MIS value in the message is 0, it is strongly recommended to release this association





&gt;&gt; Unfortunately, the SCTP implementation of Linux will be affected by the attacks described above, such as the open source lksctp.
&gt; Have you reported that issue? If not, I can let them know.

Zhouming:&nbsp; I still do not report this issue to them, I think I need disscuss it with you first, so you can let them know about it.

-------------

&gt; Please let me know how to write your name in ASCII to give you credit.

Zhouming:&nbsp; Thanks very much,&nbsp;&nbsp;it's a great honor，my English name is Zhouming.




------------------&nbsp;原始邮件&nbsp;------------------
发件人:                                                                                                                        "Michael Tuexen"                                                                                    <[e-mail]&gt;;
发送时间:&nbsp;2021年8月21日(星期六) 上午6:03
收件人:&nbsp;"一念之后★だ"<[e-mail]&gt;;
抄送:&nbsp;"tsvwg"<[e-mail]&gt;;"draft-ietf-tsvwg-2960bis"<[e-mail]&gt;;
主题:&nbsp;Re: [tsvwg] Some unclear points about rfc4960 (SCTP).



&gt; On 18. Aug 2021, at 18:46, Michael Tuexen <[e-mail]&gt; wrote:
&gt; 
&gt;&gt; On 17. Aug 2021, at 18:56, 一念之后★だ <[e-mail]&gt; wrote:
&gt;&gt; 
&gt;&gt; Dear IETF experts，
&gt;&gt; 
&gt;&gt; I read rfc4960 (SCTP) and felt that there were some unclear descriptions in the document. I hope to get your reply, so I wrote this email to you. And I think it's better to describe these more clearly in the subsequent version of the RFC.
&gt;&gt; The following description describes some scenarios requiring abort Association in rfc4960:
&gt;&gt; Page25
&gt;&gt; 
&gt;&gt; 
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; Page26
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; Page30
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; Page31
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; Page56
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; As can be seen from the above description, when an init message or init ACK message is received,
&gt;&gt; (1)If the init tag in the message is 0, the association must be released
&gt;&gt; (2)If the mandatory parameters in the message are not fully carried, the association must be released
&gt;&gt; (3)When the OS value or MIS value in the message is 0, it is strongly recommended to release this association
&gt; You are correct, using an initiate tag of 0 or limit the number of streams to 0 is not allowed.
&gt; 
&gt; Please note the in the case of an initiate tag of 0 in an INIT chunk, you can't send an ABORT
&gt; since you have no tag different from 0 you could use as the verification tag.
&gt; This has already been clarified in draft-ietf-tsvwg-rfc4960-bis-13.
&gt; 
&gt; It states now:
&gt; <t&gt;If the value of the Initiate Tag in a received INIT chunk is found to be 0,
&gt; the receiver MUST treat it as an error and silently discard the packet.</t&gt;
&gt; 
&gt; For OS = 0 or MIS = 0, the INIT chunk MUST be dropped and an ABORT chunk
&gt; SHOULD be sent. So the next revision will contain:
&gt; 
&gt; <t&gt;A receiver of an INIT chunk with the OS value set to 0 MUST discard the
&gt; packet and SHOULD send a packet in response containing an ABORT chunk and using
&gt; the Initiate Tag as the Verification Tag.
&gt; Any existing association MUST NOT be affected.</t&gt;
&gt; 
&gt; <t&gt;A receiver of an INIT chunk with the MIS value set to 0 MUST discard the
&gt; packet and SHOULD send a packet in response containing an ABORT chunk and using
&gt; the Initiate Tag as the Verification Tag.
&gt; Any existing association MUST NOT be affected.</t&gt;
&gt; 
&gt; I guess this makes the description for the INIT chunk crystal clear.
&gt; 
&gt; The situation for the INIT-ACK is different. The INIT-ACK is only processed, if
&gt; the verification tag in the common header was correct. In this case it is OK
&gt; to destroy the association. A potential attacker has to guess the 32-bit
&gt; verification tag and if the attacker can do this, the attacker can send a
&gt; packet containing an ABORT chunk.
&gt; However, the consistency of the text can be improved.
Here are the suggested changes to improve the consistency of the handling
of illegal parameters in the INIT ACK chunk:

For the Initiate Tag:
<t&gt;If an endpoint in the COOKIE-WAIT state receives an INIT ACK chunk with the
Initiate Tag set to 0, it MUST destroy the TCB and SHOULD send an ABORT chunk
with the T bit set.
If such an INIT-ACK chunk is received in any state other than CLOSED or
COOKIE-WAIT, it SHOULD be discarded silently
(see <xref target='sec_unexpected_init_ack'/&gt;).</t&gt;


For OS:
<t&gt;If an endpoint in the COOKIE-WAIT state receives an INIT ACK chunk with the
OS value set to 0, it MUST destroy the TCB and SHOULD send an ABORT chunk.
If such an INIT-ACK chunk is received in any state other than CLOSED or
COOKIE-WAIT, it SHOULD be discarded silently
(see <xref target='sec_unexpected_init_ack'/&gt;).</t&gt;

For MIS:
<t&gt;If an endpoint in the COOKIE-WAIT state receives an INIT ACK chunk with the
MIS value set to 0, it MUST destroy the TCB and SHOULD send an ABORT chunk.
If such an INIT-ACK chunk is received in any state other than CLOSED or
COOKIE-WAIT, it SHOULD be discarded silently
(see <xref target='sec_unexpected_init_ack'/&gt;).</t&gt;

Best regards
Michael
&gt;&gt; 
&gt;&gt; The following description also shows that there can only be one SCTP association between two SCTP endpoints at any time:
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; 
&gt;&gt; Because init messages can be forged arbitrarily, take the scenario where the initial tag in init message is 0 as an example.
&gt;&gt; In this scenario, a malicious attacker can destroy other associations at a very low cost:
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; For the other two scenarios, i.e. init message with missing mandatory parameters or init message with OS value / MIS value of 0, the effect may be similar.
&gt;&gt; I think from the original intention of the protocol design, these error scenarios should be handled according to the following. However, the RFC does not specify how to deal with the established association endpoint when receiving this wrong init (or init ACK) message, and whether the corresponding association should still be removed.
&gt; I agree. I think the suggested changes above make this crystal clear.
&gt;&gt; <[e-mail]&gt;
&gt;&gt; 
&gt;&gt; Unfortunately, the SCTP implementation of Linux will be affected by the attacks described above, such as the open source lksctp.
&gt; Have you reported that issue? If not, I can let them know.
&gt;&gt; I hope to receive your reply to clarify the above unclear areas.
&gt;&gt; Thank you very much.
&gt; Thank you very mich for reporting the issue.
&gt;&gt; 
&gt;&gt; Yours faithfully!
&gt;&gt; Zhouming.
&gt; Please let me know how to write your name in ASCII to give you credit.
&gt; 
&gt; Best regards
&gt; Michael
&gt;