diff options
| author | Claudiu Manoil <claudiu.manoil@freescale.com> | 2012-11-08 23:11:41 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-11-09 23:07:22 +0100 |
| commit | 1eb8f7a7da6d3b36fcc339d7bbddbf8315a5c1dd (patch) | |
| tree | 08367aa6aa5fa54af7a9d82d21a98195f050880e /drivers/isdn/hardware/mISDN/hfcpci.c | |
| parent | virtio_net: use net_*_ratelimited() helpers (diff) | |
| download | linux-1eb8f7a7da6d3b36fcc339d7bbddbf8315a5c1dd.tar.xz 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 'drivers/isdn/hardware/mISDN/hfcpci.c')
0 files changed, 0 insertions, 0 deletions