]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "net/sctp: fix race condition in sctp_destroy_sock"
authorXin Long <lucien.xin@gmail.com>
Sun, 2 May 2021 21:11:41 +0000 (05:11 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 3 May 2021 20:36:21 +0000 (13:36 -0700)
commit91e1c8bcded4b456d57cf957f8042682c0122581
tree493ce8d5208472d790b499e710c44a7704cbe256
parentacde5a9ed9bafafa86c2bac949f40bcc65f352a3
Revert "net/sctp: fix race condition in sctp_destroy_sock"

This reverts commit 30058c4163660a7f4f8fc953b7f9f976ed5ecf99.

This one has to be reverted as it introduced a dead lock, as
syzbot reported:

       CPU0                    CPU1
       ----                    ----
  lock(&net->sctp.addr_wq_lock);
                               lock(slock-AF_INET6);
                               lock(&net->sctp.addr_wq_lock);
  lock(slock-AF_INET6);

CPU0 is the thread of sctp_addr_wq_timeout_handler(), and CPU1
is that of sctp_close().

The original issue this commit fixed will be fixed in the next
patch.

Reported-by: syzbot+959223586843e69a2674@syzkaller.appspotmail.com
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sctp/socket.c