summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFelix Fietkau <nbd@openwrt.org>2015-12-23 11:40:53 +0000
committerFelix Fietkau <nbd@openwrt.org>2015-12-23 11:40:53 +0000
commit10afb1bd8c3af30b88fa1c09dde91b1e8a8cf27c (patch)
tree8da5e941025e2e5f6ebc03e4859313f297be2710
parentdbf2ec47a6d94f5899fc3f681f2384ce53526779 (diff)
downloadmtk-20170518-10afb1bd8c3af30b88fa1c09dde91b1e8a8cf27c.zip
mtk-20170518-10afb1bd8c3af30b88fa1c09dde91b1e8a8cf27c.tar.gz
mtk-20170518-10afb1bd8c3af30b88fa1c09dde91b1e8a8cf27c.tar.bz2
kernel: add remaining kernel patch accidentally left out of r47963
Signed-off-by: Felix Fietkau <nbd@openwrt.org> SVN-Revision: 47966
-rw-r--r--target/linux/generic/patches-3.18/081-06-ppp-don-t-set-sk_state-to-PPPOX_ZOMBIE-in-pppoe_disc.patch45
1 files changed, 45 insertions, 0 deletions
diff --git a/target/linux/generic/patches-3.18/081-06-ppp-don-t-set-sk_state-to-PPPOX_ZOMBIE-in-pppoe_disc.patch b/target/linux/generic/patches-3.18/081-06-ppp-don-t-set-sk_state-to-PPPOX_ZOMBIE-in-pppoe_disc.patch
index e69de29..8d155eb 100644
--- a/target/linux/generic/patches-3.18/081-06-ppp-don-t-set-sk_state-to-PPPOX_ZOMBIE-in-pppoe_disc.patch
+++ b/target/linux/generic/patches-3.18/081-06-ppp-don-t-set-sk_state-to-PPPOX_ZOMBIE-in-pppoe_disc.patch
@@ -0,0 +1,45 @@
+From: Guillaume Nault <g.nault@alphalink.fr>
+Date: Thu, 19 Nov 2015 12:52:56 +0100
+Subject: [PATCH] ppp: don't set sk_state to PPPOX_ZOMBIE in pppoe_disc_rcv()
+
+Since 287f3a943fef ("pppoe: Use workqueue to die properly when a PADT
+is received"), pppoe_disc_rcv() disconnects the socket by scheduling
+pppoe_unbind_sock_work(). This is enough to stop socket transmission
+and makes the PPPOX_ZOMBIE state uncessary.
+
+Signed-off-by: Guillaume Nault <g.nault@alphalink.fr>
+Signed-off-by: David S. Miller <davem@davemloft.net>
+---
+
+--- a/drivers/net/ppp/pppoe.c
++++ b/drivers/net/ppp/pppoe.c
+@@ -500,27 +500,9 @@ static int pppoe_disc_rcv(struct sk_buff
+
+ pn = pppoe_pernet(dev_net(dev));
+ po = get_item(pn, ph->sid, eth_hdr(skb)->h_source, dev->ifindex);
+- if (po) {
+- struct sock *sk = sk_pppox(po);
+-
+- bh_lock_sock(sk);
+-
+- /* If the user has locked the socket, just ignore
+- * the packet. With the way two rcv protocols hook into
+- * one socket family type, we cannot (easily) distinguish
+- * what kind of SKB it is during backlog rcv.
+- */
+- if (sock_owned_by_user(sk) == 0) {
+- /* We're no longer connect at the PPPOE layer,
+- * and must wait for ppp channel to disconnect us.
+- */
+- sk->sk_state = PPPOX_ZOMBIE;
+- }
+-
+- bh_unlock_sock(sk);
++ if (po)
+ if (!schedule_work(&po->proto.pppoe.padt_work))
+- sock_put(sk);
+- }
++ sock_put(sk_pppox(po));
+
+ abort:
+ kfree_skb(skb);