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/target | |
parent | be44389964c1855ccffedbfbf5d2a943bb84ad66 (diff) |
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/target')
0 files changed, 0 insertions, 0 deletions