dccp: check sk for closed state in dccp_sendmsg()
authorAlexey Kodanev <alexey.kodanev@oracle.com>
Tue, 6 Mar 2018 19:57:01 +0000 (22:57 +0300)
committerBen Hutchings <ben@decadent.org.uk>
Thu, 31 May 2018 23:30:01 +0000 (00:30 +0100)
commit109503b8cccb3b803d875b88d21d49eab921969e
tree095e9a73eeda22434643b34889f76ded7af4d1c4
parent02a37ffd681be59775c9f13686e20621f7097f7e
dccp: check sk for closed state in dccp_sendmsg()

commit 67f93df79aeefc3add4e4b31a752600f834236e2 upstream.

dccp_disconnect() sets 'dp->dccps_hc_tx_ccid' tx handler to NULL,
therefore if DCCP socket is disconnected and dccp_sendmsg() is
called after it, it will cause a NULL pointer dereference in
dccp_write_xmit().

This crash and the reproducer was reported by syzbot. Looks like
it is reproduced if commit 69c64866ce07 ("dccp: CVE-2017-8824:
use-after-free in DCCP code") is applied.

Reported-by: syzbot+f99ab3887ab65d70f816@syzkaller.appspotmail.com
Signed-off-by: Alexey Kodanev <alexey.kodanev@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
net/dccp/proto.c