You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
30 lines
1.1 KiB
Diff
30 lines
1.1 KiB
Diff
From: Guillaume Nault <g.nault@alphalink.fr>
|
|
Date: Thu, 22 Oct 2015 16:57:10 +0200
|
|
Subject: [PATCH] ppp: fix pppoe_dev deletion condition in pppoe_release()
|
|
|
|
We can't rely on PPPOX_ZOMBIE to decide whether to clear po->pppoe_dev.
|
|
PPPOX_ZOMBIE can be set by pppoe_disc_rcv() even when po->pppoe_dev is
|
|
NULL. So we have no guarantee that (sk->sk_state & PPPOX_ZOMBIE) implies
|
|
(po->pppoe_dev != NULL).
|
|
Since we're releasing a PPPoE socket, we want to release the pppoe_dev
|
|
if it exists and reset sk_state to PPPOX_DEAD, no matter the previous
|
|
value of sk_state. So we can just check for po->pppoe_dev and avoid any
|
|
assumption on sk->sk_state.
|
|
|
|
Fixes: 2b018d57ff18 ("pppoe: drop PPPOX_ZOMBIEs in pppoe_release")
|
|
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
|
|
@@ -589,7 +589,7 @@ static int pppoe_release(struct socket *
|
|
|
|
po = pppox_sk(sk);
|
|
|
|
- if (sk->sk_state & (PPPOX_CONNECTED | PPPOX_BOUND | PPPOX_ZOMBIE)) {
|
|
+ if (po->pppoe_dev) {
|
|
dev_put(po->pppoe_dev);
|
|
po->pppoe_dev = NULL;
|
|
}
|