Staging: lustre: llite: dir: remove unneeded null test before free
authorJulia Lawall <Julia.Lawall@lip6.fr>
Fri, 1 May 2015 19:38:02 +0000 (21:38 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 8 May 2015 07:23:52 +0000 (09:23 +0200)
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>

This patch additionally simplifies one case where the free, and thus the
jump to the end of the function, is not needed.

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/llite/dir.c

index f17154aaa3297cf28bc09bdddfec3429bc41bc87..4b0de8d89e5c54d306f671cae9f51d4c3c6c540d 100644 (file)
@@ -754,10 +754,8 @@ int ll_dir_setstripe(struct inode *inode, struct lov_user_md *lump,
                char *buf;
 
                param = kzalloc(MGS_PARAM_MAXLEN, GFP_NOFS);
-               if (!param) {
-                       rc = -ENOMEM;
-                       goto end;
-               }
+               if (!param)
+                       return -ENOMEM;
 
                buf = param;
                /* Get fsname and assume devname to be -MDT0000. */
@@ -786,8 +784,7 @@ int ll_dir_setstripe(struct inode *inode, struct lov_user_md *lump,
                rc = ll_send_mgc_param(mgc->u.cli.cl_mgc_mgsexp, param);
 
 end:
-               if (param != NULL)
-                       kfree(param);
+               kfree(param);
        }
        return rc;
 }