diff options
| author | Patrick McHardy <kaber@trash.net> | 2007-03-29 14:46:52 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2007-03-29 14:46:52 -0400 |
| commit | c01003c20563d1e75ec9828d21743919d2b43977 (patch) | |
| tree | 21cae8933e8a4908d8e8c24244a627bf0c997e77 /include/linux/skbuff.h | |
| parent | db8b22550d4b83f0910d27a34d05aa16f7f7159f (diff) | |
[IFB]: Fix crash on input device removal
The input_device pointer is not refcounted, which means the device may
disappear while packets are queued, causing a crash when ifb passes packets
with a stale skb->dev pointer to netif_rx().
Fix by storing the interface index instead and do a lookup where neccessary.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Acked-by: Jamal Hadi Salim <hadi@cyberus.ca>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/skbuff.h')
| -rw-r--r-- | include/linux/skbuff.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 4ff3940210d8..82f43ad478c7 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h | |||
| @@ -188,7 +188,7 @@ enum { | |||
| 188 | * @sk: Socket we are owned by | 188 | * @sk: Socket we are owned by |
| 189 | * @tstamp: Time we arrived | 189 | * @tstamp: Time we arrived |
| 190 | * @dev: Device we arrived on/are leaving by | 190 | * @dev: Device we arrived on/are leaving by |
| 191 | * @input_dev: Device we arrived on | 191 | * @iif: ifindex of device we arrived on |
| 192 | * @h: Transport layer header | 192 | * @h: Transport layer header |
| 193 | * @nh: Network layer header | 193 | * @nh: Network layer header |
| 194 | * @mac: Link layer header | 194 | * @mac: Link layer header |
| @@ -235,7 +235,8 @@ struct sk_buff { | |||
| 235 | struct sock *sk; | 235 | struct sock *sk; |
| 236 | struct skb_timeval tstamp; | 236 | struct skb_timeval tstamp; |
| 237 | struct net_device *dev; | 237 | struct net_device *dev; |
| 238 | struct net_device *input_dev; | 238 | int iif; |
| 239 | /* 4 byte hole on 64 bit*/ | ||
| 239 | 240 | ||
| 240 | union { | 241 | union { |
| 241 | struct tcphdr *th; | 242 | struct tcphdr *th; |
