diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2008-12-05 19:03:09 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2009-01-06 11:53:53 -0500 |
commit | 05f3a9af58180d24a9decedd71d4587935782d70 (patch) | |
tree | c5c53a1aa8ba8fdf47f2613970c85ec0a4b0d63d /fs/lockd/mon.c | |
parent | bc1cc6c4e476b60df48227165990c87a22db6bb7 (diff) |
NSM: Remove !nsm check from nsm_release()
The nsm_release() function should never be called with a NULL handle
point. If it is, that's a bug.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs/lockd/mon.c')
-rw-r--r-- | fs/lockd/mon.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/lockd/mon.c b/fs/lockd/mon.c index 0a066a13478e..0792900b6281 100644 --- a/fs/lockd/mon.c +++ b/fs/lockd/mon.c | |||
@@ -288,8 +288,6 @@ found: | |||
288 | */ | 288 | */ |
289 | void nsm_release(struct nsm_handle *nsm) | 289 | void nsm_release(struct nsm_handle *nsm) |
290 | { | 290 | { |
291 | if (!nsm) | ||
292 | return; | ||
293 | if (atomic_dec_and_lock(&nsm->sm_count, &nsm_lock)) { | 291 | if (atomic_dec_and_lock(&nsm->sm_count, &nsm_lock)) { |
294 | list_del(&nsm->sm_link); | 292 | list_del(&nsm->sm_link); |
295 | spin_unlock(&nsm_lock); | 293 | spin_unlock(&nsm_lock); |