diff options
author | Himangi Saraogi <himangi774@gmail.com> | 2014-07-19 15:34:05 +0400 |
---|---|---|
committer | Richard Weinberger <richard@nod.at> | 2014-10-13 23:46:26 +0400 |
commit | 5f786595a291092d20fafe10c5a30378971a8cc3 (patch) | |
tree | c733a77931002ab271566ba81f6b926ae240b531 | |
parent | 970e51feaddbc33ed0e7d187af7f69d1a12c7b6a (diff) | |
download | linux-5f786595a291092d20fafe10c5a30378971a8cc3.tar.xz |
um: net: Eliminate NULL test after alloc_bootmem
alloc_bootmem and related functions never return NULL. Thus a NULL
test or memset after calls to these functions is unnecessary.
The following Coccinelle semantic patch was used for making the change:
@@
expression E;
statement S;
@@
E = \(alloc_bootmem\|alloc_bootmem_low\|alloc_bootmem_pages\|alloc_bootmem_low_pages\)(...)
... when != E
- if (E == NULL) S
Signed-off-by: Himangi Saraogi <himangi774@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Richard Weinberger <richard@nod.at>
-rw-r--r-- | arch/um/drivers/net_kern.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/arch/um/drivers/net_kern.c b/arch/um/drivers/net_kern.c index 7d26d9c0b2fb..f70dd540655d 100644 --- a/arch/um/drivers/net_kern.c +++ b/arch/um/drivers/net_kern.c @@ -659,10 +659,6 @@ static int __init eth_setup(char *str) } new = alloc_bootmem(sizeof(*new)); - if (new == NULL) { - printk(KERN_ERR "eth_init : alloc_bootmem failed\n"); - return 1; - } INIT_LIST_HEAD(&new->list); new->index = n; |