diff options
| author | Thomas Meyer <thomas@m3y3r.de> | 2018-12-03 10:02:01 -0600 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2018-12-03 10:02:01 -0600 |
| commit | 3456880ff39f7606866b5852fd71ef6663200c0d (patch) | |
| tree | 7ee2a3ff6264db1181d54f751b3ad8172f7f7e8d /fs/dlm/lockspace.c | |
| parent | f31a89692830061bceba8469607e4e4b0f900159 (diff) | |
| download | lwn-3456880ff39f7606866b5852fd71ef6663200c0d.tar.gz lwn-3456880ff39f7606866b5852fd71ef6663200c0d.zip | |
dlm: NULL check before some freeing functions is not needed
NULL check before some freeing functions is not needed.
Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/lockspace.c')
| -rw-r--r-- | fs/dlm/lockspace.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/dlm/lockspace.c b/fs/dlm/lockspace.c index a0584bc5209a..db43b98c4d64 100644 --- a/fs/dlm/lockspace.c +++ b/fs/dlm/lockspace.c @@ -681,10 +681,8 @@ static int new_lockspace(const char *name, const char *cluster, out_lkbidr: idr_destroy(&ls->ls_lkbidr); out_rsbtbl: - 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]); vfree(ls->ls_rsbtbl); out_lsfree: if (do_unreg) |