dccp: CVE-2017-8824

dccp: CVE-2017-8824: use-after-free in DCCP code

Reference:
https://nvd.nist.gov/vuln/detail/CVE-2017-8824
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-4.14.y&id=e5e1e153ecd40c2b09c23d2b2c161a6c8ddbba07

Change-Id: Ic8d564518bde1ca36f0b9c54f1fa603e8778c92b
Signed-off-by: Andreas Wellving <andreas.wellving@enea.com>

1 files changed, 49 insertions, 0 deletions

diff --git a/patches/cve/CVE-2017-8824-dccp-CVE-2017-8824-use-after-free-in-DCCP-code.patch b/patches/cve/CVE-2017-8824-dccp-CVE-2017-8824-use-after-free-in-DCCP-code.patch
new file mode 100644
index 0000000..964e1c1
--- /dev/null
+++ b/patches/cve/CVE-2017-8824-dccp-CVE-2017-8824-use-after-free-in-DCCP-code.patch
@@ -0,0 +1,49 @@
+From e5e1e153ecd40c2b09c23d2b2c161a6c8ddbba07 Mon Sep 17 00:00:00 2001
+From: Mohamed Ghannam <simo.ghannam@gmail.com>
+Date: Tue, 5 Dec 2017 20:58:35 +0000
+Subject: [PATCH] dccp: CVE-2017-8824: use-after-free in DCCP code
+
+commit 69c64866ce072dea1d1e59a0d61e0f66c0dffb76 upstream.
+
+Whenever the sock object is in DCCP_CLOSED state,
+dccp_disconnect() must free dccps_hc_tx_ccid and
+dccps_hc_rx_ccid and set to NULL.
+
+CVE: CVE-2017-8824
+Upstream-Status: Backport [https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-4.14.y&id=e5e1e153ecd40c2b09c23d2b2c161a6c8ddbba07]
+
+Signed-off-by: Mohamed Ghannam <simo.ghannam@gmail.com>
+Reviewed-by: Eric Dumazet <edumazet@google.com>
+Signed-off-by: David S. Miller <davem@davemloft.net>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+Signed-off-by: Andreas Wellving <andreas.wellving@enea.com>
+---
+ net/dccp/proto.c | 5 +++++
+ 1 file changed, 5 insertions(+)
+
+diff --git a/net/dccp/proto.c b/net/dccp/proto.c
+index b68168fcc06a..9d43c1f40274 100644
+--- a/net/dccp/proto.c
++++ b/net/dccp/proto.c
+@@ -259,6 +259,7 @@ int dccp_disconnect(struct sock *sk, int flags)
+ {
+        struct inet_connection_sock *icsk = inet_csk(sk);
+        struct inet_sock *inet = inet_sk(sk);
++       struct dccp_sock *dp = dccp_sk(sk);
+        int err = 0;
+        const int old_state = sk->sk_state;
+ 
+@@ -278,6 +279,10 @@ int dccp_disconnect(struct sock *sk, int flags)
+                sk->sk_err = ECONNRESET;
+ 
+        dccp_clear_xmit_timers(sk);
++       ccid_hc_rx_delete(dp->dccps_hc_rx_ccid, sk);
++       ccid_hc_tx_delete(dp->dccps_hc_tx_ccid, sk);
++       dp->dccps_hc_rx_ccid = NULL;
++       dp->dccps_hc_tx_ccid = NULL;
+ 
+        __skb_queue_purge(&sk->sk_receive_queue);
+        __skb_queue_purge(&sk->sk_write_queue);
+-- 
+2.20.1
+