diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2015-12-20 11:15:52 (GMT) |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2015-12-30 09:34:13 (GMT) |
commit | 8cb708f3d35e78f00528caf2f681900d2c7883b8 (patch) | |
tree | 0f85dd1a1582d9a2db90cf1732f8817038bc5ac1 | |
parent | 11b64c8acca05d7e50a873e0e8758b75d6d6650f (diff) | |
download | linux-8cb708f3d35e78f00528caf2f681900d2c7883b8.tar.xz |
s390/cio: add NULL test
Add NULL test on call to kzalloc.
The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression x;
identifier fld;
@@
* x = kzalloc(...);
... when != x == NULL
x->fld
// </smpl>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
-rw-r--r-- | drivers/s390/char/con3215.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/s390/char/con3215.c b/drivers/s390/char/con3215.c index 0fc3fe5..7d82bbc 100644 --- a/drivers/s390/char/con3215.c +++ b/drivers/s390/char/con3215.c @@ -922,6 +922,8 @@ static int __init con3215_init(void) spin_lock_init(&raw3215_freelist_lock); for (i = 0; i < NR_3215_REQ; i++) { req = kzalloc(sizeof(struct raw3215_req), GFP_KERNEL | GFP_DMA); + if (!req) + return -ENOMEM; req->next = raw3215_freelist; raw3215_freelist = req; } |