summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJulia Lawall <Julia.Lawall@lip6.fr>2015-05-01 19:37:56 (GMT)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2015-05-08 07:23:54 (GMT)
commit6fa7cbab38ee64eb78376a32a57cbaa074d766e4 (patch)
tree92ec67e9a78121a8f707b889f99609c1eb35b316
parent34b8fbd6cdb1251d7f2bb3645124148365c5089a (diff)
downloadlinux-6fa7cbab38ee64eb78376a32a57cbaa074d766e4.tar.xz
staging: lustre: mdc: remove unneeded null test before free
Kfree can cope with a null argument, so drop null tests. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression ptr; @@ - if (ptr != NULL) kfree(ptr); // </smpl> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/staging/lustre/lustre/mdc/mdc_request.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/lustre/lustre/mdc/mdc_request.c b/drivers/staging/lustre/lustre/mdc/mdc_request.c
index 1a4ef9d..cbbdfce 100644
--- a/drivers/staging/lustre/lustre/mdc/mdc_request.c
+++ b/drivers/staging/lustre/lustre/mdc/mdc_request.c
@@ -1644,8 +1644,7 @@ out:
llog_cat_close(NULL, llh);
if (ctxt)
llog_ctxt_put(ctxt);
- if (cs->cs_buf)
- kfree(cs->cs_buf);
+ kfree(cs->cs_buf);
kfree(cs);
return rc;
}