diff options
author | Joakim Tjernlund <Joakim.Tjernlund@transmode.se> | 2015-04-20 15:07:48 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-04-20 15:26:57 -0400 |
commit | a068833bbf0ede9203410f8fd186178c758ade9b (patch) | |
tree | 4ed6dc262885a8137d7871ac12f8a74cb5d32ac7 /drivers/net | |
parent | 2ab957492d13bb819400ac29ae55911d50a82a13 (diff) |
pppoe: Lacks DST MAC address check
A pppoe session is identified by its session ID and MAC address.
Currently pppoe does not check if the received pkg has the correct
MAC address. This is a problem when the eth I/F is in promisc mode
as then any DST MAC address is accepted.
Signed-off-by: Joakim Tjernlund <joakim.tjernlund@transmode.se>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/ppp/pppoe.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/net/ppp/pppoe.c b/drivers/net/ppp/pppoe.c index ff059e1d8ac6..aa1dd926623a 100644 --- a/drivers/net/ppp/pppoe.c +++ b/drivers/net/ppp/pppoe.c | |||
@@ -380,6 +380,9 @@ static int pppoe_rcv_core(struct sock *sk, struct sk_buff *skb) | |||
380 | * can't change. | 380 | * can't change. |
381 | */ | 381 | */ |
382 | 382 | ||
383 | if (skb->pkt_type == PACKET_OTHERHOST) | ||
384 | goto abort_kfree; | ||
385 | |||
383 | if (sk->sk_state & PPPOX_BOUND) { | 386 | if (sk->sk_state & PPPOX_BOUND) { |
384 | ppp_input(&po->chan, skb); | 387 | ppp_input(&po->chan, skb); |
385 | } else if (sk->sk_state & PPPOX_RELAY) { | 388 | } else if (sk->sk_state & PPPOX_RELAY) { |