From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Date: Tue, 13 Sep 2016 22:15:56 +0200 The check for the local variable "table" is unnecessary at the end of this function because the corresponding source code place should only be reached with a non-zero pointer for it after a failed call of the function "kmalloc_array". Thus remove it. Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> --- drivers/char/agp/uninorth-agp.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/char/agp/uninorth-agp.c b/drivers/char/agp/uninorth-agp.c index 8b1eb72..657cf17 100644 --- a/drivers/char/agp/uninorth-agp.c +++ b/drivers/char/agp/uninorth-agp.c @@ -438,8 +438,7 @@ static int uninorth_create_gatt_table(struct agp_bridge_data *bridge) return 0; free_page_array: kfree(uninorth_priv.pages_arr); - if (table) - free_pages((unsigned long)table, page_order); + free_pages((unsigned long)table, page_order); return -ENOMEM; } -- 2.10.0 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html