diff options
| author |   Claudiu Manoil <claudiu.manoil@freescale.com> | 2012-11-08 22:11:41 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2012-11-09 17:07:22 -0500 |
| commit | 1eb8f7a7da6d3b36fcc339d7bbddbf8315a5c1dd (patch) | |
| tree | 08367aa6aa5fa54af7a9d82d21a98195f050880e /include/uapi/linux/if_tunnel.h | |
| parent | virtio_net: use net_*_ratelimited() helpers (diff) | |
| download | wireguard-linux-1eb8f7a7da6d3b36fcc339d7bbddbf8315a5c1dd.tar.xz wireguard-linux-1eb8f7a7da6d3b36fcc339d7bbddbf8315a5c1dd.zip | |
gianfar: Fix alloc_skb_resources on -ENOMEM cleanup path
Should gfar_init_bds() return with -ENOMEM inside gfar_alloc_skb_resources(),
free_skb_resources() will be called twice in a row on the "cleanup" path,
leading to duplicate kfree() calls for rx_|tx_queue->rx_|tx_skbuff resulting
in segmentation fault.
This patch prevents the segmentation fault to happen in the future
(rx_|tx_sbkbuff set to NULL), and corrects the error path handling
for gfar_init_bds().
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: Claudiu Manoil <claudiu.manoil@freescale.com>
Acked-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/uapi/linux/if_tunnel.h')
0 files changed, 0 insertions, 0 deletions