diff options
| author | Patrick McHardy <kaber@trash.net> | 2007-10-18 05:09:28 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2007-10-18 05:09:28 -0700 |
| commit | 1b83336bb9fa4f110b5b0a4378a82cd5f0fd0ac8 (patch) | |
| tree | 0eb85178fe55ac084d2e62037432a1f8ae48b74d /include/linux/netdevice.h | |
| parent | df2e014bfbbf26d160e1bf47b05fc05c96205e2b (diff) | |
| download | lwn-1b83336bb9fa4f110b5b0a4378a82cd5f0fd0ac8.tar.gz lwn-1b83336bb9fa4f110b5b0a4378a82cd5f0fd0ac8.zip | |
[NET]: Fix OOPS due to missing check in dev_parse_header().
[ This is kernel bugzilla 9174 "linux-2.6.23-git11 kernel panic" ]
The device in question is an IPv6-over-IPv4 tunnel, which doesn't have
any header_ops, so the crash happens in dev_parse_header when
dereferencing them.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/netdevice.h')
| -rw-r--r-- | include/linux/netdevice.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index 39dd83b183a9..452c88d971ad 100644 --- a/include/linux/netdevice.h +++ b/include/linux/netdevice.h @@ -827,7 +827,7 @@ static inline int dev_parse_header(const struct sk_buff *skb, { const struct net_device *dev = skb->dev; - if (!dev->header_ops->parse) + if (!dev->header_ops || !dev->header_ops->parse) return 0; return dev->header_ops->parse(skb, haddr); } |