No need of NULL check before kfree,
removing the same.

Signed-off-by: Maninder Singh <maninder...@samsung.com>
Reviewed-by: Vaneet Narang <v.nar...@samsung.com>
---
 fs/dlm/lockspace.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/fs/dlm/lockspace.c b/fs/dlm/lockspace.c
index f3e7278..41c53b3 100644
--- a/fs/dlm/lockspace.c
+++ b/fs/dlm/lockspace.c
@@ -673,10 +673,8 @@ static int new_lockspace(const char *name, const char 
*cluster,
        kfree(ls->ls_recover_buf);
  out_lkbidr:
        idr_destroy(&ls->ls_lkbidr);
-       for (i = 0; i < DLM_REMOVE_NAMES_MAX; i++) {
-               if (ls->ls_remove_names[i])
-                       kfree(ls->ls_remove_names[i]);
-       }
+       for (i = 0; i < DLM_REMOVE_NAMES_MAX; i++)
+               kfree(ls->ls_remove_names[i]);
  out_rsbtbl:
        vfree(ls->ls_rsbtbl);
  out_lsfree:
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to