diff options
| author | Ang Way Chuang <wcang@nav6.org> | 2008-06-18 00:10:33 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-06-18 00:10:33 -0400 |
| commit | f09f7ee20c867818bacf79426cf491b2749e7eff (patch) | |
| tree | c570f7385827241c45dae25611759b0909ede9ce | |
| parent | fe833fca2eac6b3d3ad5e35f44ad4638362f1da8 (diff) | |
tun: Proper handling of IPv6 header in tun driver when TUN_NO_PI is set
By default, tun.c running in TUN_TUN_DEV mode will set the protocol of
packet to IPv4 if TUN_NO_PI is set. My program failed to work when I
assumed that the driver will check the first nibble of packet,
determine IP version and set the appropriate protocol.
Signed-off-by: Ang Way Chuang <wcang@nav6.org>
Acked-by: Max Krasnyansky <maxk@qualcomm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/tun.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/drivers/net/tun.c b/drivers/net/tun.c index 0ce07a339c7e..7ab94c825b57 100644 --- a/drivers/net/tun.c +++ b/drivers/net/tun.c | |||
| @@ -313,6 +313,21 @@ static __inline__ ssize_t tun_get_user(struct tun_struct *tun, struct iovec *iv, | |||
| 313 | 313 | ||
| 314 | switch (tun->flags & TUN_TYPE_MASK) { | 314 | switch (tun->flags & TUN_TYPE_MASK) { |
| 315 | case TUN_TUN_DEV: | 315 | case TUN_TUN_DEV: |
| 316 | if (tun->flags & TUN_NO_PI) { | ||
| 317 | switch (skb->data[0] & 0xf0) { | ||
| 318 | case 0x40: | ||
| 319 | pi.proto = htons(ETH_P_IP); | ||
| 320 | break; | ||
| 321 | case 0x60: | ||
| 322 | pi.proto = htons(ETH_P_IPV6); | ||
| 323 | break; | ||
| 324 | default: | ||
| 325 | tun->dev->stats.rx_dropped++; | ||
| 326 | kfree_skb(skb); | ||
| 327 | return -EINVAL; | ||
| 328 | } | ||
| 329 | } | ||
| 330 | |||
| 316 | skb_reset_mac_header(skb); | 331 | skb_reset_mac_header(skb); |
| 317 | skb->protocol = pi.proto; | 332 | skb->protocol = pi.proto; |
| 318 | skb->dev = tun->dev; | 333 | skb->dev = tun->dev; |
