summaryrefslogtreecommitdiffstats
path: root/drivers/infiniband
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2017-07-13 10:45:48 +0300
committerDoug Ledford <dledford@redhat.com>2017-07-20 11:20:49 -0400
commit5c8857b653e71a9850a02837e1268e3198abbd1a (patch)
treedd3479ff4bcc74152fb5ade330fa4e062467fe53 /drivers/infiniband
parent653f0a71daf1a71d55d3af368c28c8114c11c607 (diff)
downloadlinux-5c8857b653e71a9850a02837e1268e3198abbd1a.tar.gz
linux-5c8857b653e71a9850a02837e1268e3198abbd1a.tar.bz2
linux-5c8857b653e71a9850a02837e1268e3198abbd1a.zip
IB/IPoIB: Fix error code in ipoib_add_port()
We accidentally don't see the error code on some of these error paths. It means we return ERR_PTR(0) which is NULL and it results in a NULL dereference in the caller. This bug dates to pre-git days. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Leon Romanovsky <leonro@mellanox.com> Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r--drivers/infiniband/ulp/ipoib/ipoib_main.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c
index 70dacaf9044e..4ce315c92b48 100644
--- a/drivers/infiniband/ulp/ipoib/ipoib_main.c
+++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c
@@ -2239,6 +2239,7 @@ static struct net_device *ipoib_add_port(const char *format,
goto register_failed;
}
+ result = -ENOMEM;
if (ipoib_cm_add_mode_attr(priv->dev))
goto sysfs_failed;
if (ipoib_add_pkey_attr(priv->dev))