diff options
author | Peter Oberparleiter <peter.oberparleiter@de.ibm.com> | 2006-02-01 06:06:35 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-02-01 11:53:24 -0500 |
commit | 600b5d163d7434332539cac7be00f649cbbfedcc (patch) | |
tree | 3a999e6576a8593bf8b49d1e768f7e990eb78a3a /drivers/s390/char/con3270.c | |
parent | 3633b0475eb269dcb85923b23b321505f06b9488 (diff) |
[PATCH] s390: ccw_device_probe_console return value
The return code of ccw_device_probe_console() is not properly handled. It
should only return a valid ccw device pointer or a error value converted by
ERR_PTR. Fix the console driver code to check with IS_ERR instead against
NULL.
Signed-off-by: Peter Oberparleiter <peter.oberparleiter@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/s390/char/con3270.c')
-rw-r--r-- | drivers/s390/char/con3270.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/s390/char/con3270.c b/drivers/s390/char/con3270.c index c570a9f6ce9c..ef607a1de55a 100644 --- a/drivers/s390/char/con3270.c +++ b/drivers/s390/char/con3270.c | |||
@@ -15,6 +15,7 @@ | |||
15 | #include <linux/interrupt.h> | 15 | #include <linux/interrupt.h> |
16 | #include <linux/list.h> | 16 | #include <linux/list.h> |
17 | #include <linux/types.h> | 17 | #include <linux/types.h> |
18 | #include <linux/err.h> | ||
18 | 19 | ||
19 | #include <asm/ccwdev.h> | 20 | #include <asm/ccwdev.h> |
20 | #include <asm/cio.h> | 21 | #include <asm/cio.h> |
@@ -597,7 +598,7 @@ con3270_init(void) | |||
597 | } | 598 | } |
598 | 599 | ||
599 | cdev = ccw_device_probe_console(); | 600 | cdev = ccw_device_probe_console(); |
600 | if (!cdev) | 601 | if (IS_ERR(cdev)) |
601 | return -ENODEV; | 602 | return -ENODEV; |
602 | rp = raw3270_setup_console(cdev); | 603 | rp = raw3270_setup_console(cdev); |
603 | if (IS_ERR(rp)) | 604 | if (IS_ERR(rp)) |