From: Itai Garbi Date: Tue, 19 Feb 2013 15:40:24 +0000 (+0000) Subject: IPoIB: Don't attempt to release resources on error flow X-Git-Tag: upstream/snapshot3+hdmi~5641^2^3~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5a2815f03c0fd5a091c95af93b7f1a17a971ac20;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git IPoIB: Don't attempt to release resources on error flow If the ipoib client info isn't found on the _remove_one callback, we must not attempt to scan the returned null list. Found by Coverity. Signed-off-by: Itai Garbi Signed-off-by: Or Gerlitz Signed-off-by: Roland Dreier --- diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index 4fe44eb..66d6da9 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -1693,6 +1693,8 @@ static void ipoib_remove_one(struct ib_device *device) return; dev_list = ib_get_client_data(device, &ipoib_client); + if (!dev_list) + return; list_for_each_entry_safe(priv, tmp, dev_list, list) { ib_unregister_event_handler(&priv->event_handler);