diff options
author | Jani Nikula <ext-jani.1.nikula@nokia.com> | 2010-03-11 11:17:28 -0500 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2010-03-31 17:26:52 -0400 |
commit | 3e340c05c0def3bb68db6751299b7821c2ba0621 (patch) | |
tree | 80842344ef8689c8e44307bedbfa719fbb30322a /drivers | |
parent | b72c40949b0f04728f2993a1434598d3bad094ea (diff) |
IB/cm: Fix device_create() return value check
Use IS_ERR() instead of comparing to NULL.
Signed-off-by: Jani Nikula <ext-jani.1.nikula@nokia.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/core/cm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/core/cm.c b/drivers/infiniband/core/cm.c index 764787ebe8d8..c9730cb3f695 100644 --- a/drivers/infiniband/core/cm.c +++ b/drivers/infiniband/core/cm.c | |||
@@ -3693,7 +3693,7 @@ static void cm_add_one(struct ib_device *ib_device) | |||
3693 | cm_dev->device = device_create(&cm_class, &ib_device->dev, | 3693 | cm_dev->device = device_create(&cm_class, &ib_device->dev, |
3694 | MKDEV(0, 0), NULL, | 3694 | MKDEV(0, 0), NULL, |
3695 | "%s", ib_device->name); | 3695 | "%s", ib_device->name); |
3696 | if (!cm_dev->device) { | 3696 | if (IS_ERR(cm_dev->device)) { |
3697 | kfree(cm_dev); | 3697 | kfree(cm_dev); |
3698 | return; | 3698 | return; |
3699 | } | 3699 | } |