summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAdrian Bunk <bunk@stusta.de>2005-11-27 00:37:36 +0100
committerLinus Torvalds <torvalds@g5.osdl.org>2005-11-27 20:23:13 -0800
commit2012a116d9e6803fb072d0cfe1aae0cc4e6d6416 (patch)
treebb9d851ae660c92edb710909746eb414c8d586e0
parent16a631733135a625dd8bc83207553aaf34c0a72c (diff)
downloadop-kernel-dev-2012a116d9e6803fb072d0cfe1aae0cc4e6d6416.zip
op-kernel-dev-2012a116d9e6803fb072d0cfe1aae0cc4e6d6416.tar.gz
[PATCH] drivers/infiniband/core/mad.c: fix use-after-release case
The Coverity checker spotted this obvious use-after-release bug caused by a wrong order of the cleanups. Signed-off-by: Adrian Bunk <bunk@stusta.de> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r--drivers/infiniband/core/mad.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/core/mad.c b/drivers/infiniband/core/mad.c
index 41d6b40..d393b50 100644
--- a/drivers/infiniband/core/mad.c
+++ b/drivers/infiniband/core/mad.c
@@ -355,9 +355,9 @@ error4:
spin_unlock_irqrestore(&port_priv->reg_lock, flags);
kfree(reg_req);
error3:
- kfree(mad_agent_priv);
-error2:
ib_dereg_mr(mad_agent_priv->agent.mr);
+error2:
+ kfree(mad_agent_priv);
error1:
return ret;
}
OpenPOWER on IntegriCloud