summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2008-07-15 12:35:20 -0400
committerJ. Bruce Fields <bfields@citi.umich.edu>2008-07-15 12:35:20 -0400
commitb0e92aae157cbf78ede7b7dd03967b49fcb2102e (patch)
tree8540a64fdf71092789d994f258cb2d07054a42f1
parentafc1246f917c664b0df98b3c22fa62db74d2ca33 (diff)
downloadop-kernel-dev-b0e92aae157cbf78ede7b7dd03967b49fcb2102e.zip
op-kernel-dev-b0e92aae157cbf78ede7b7dd03967b49fcb2102e.tar.gz
lockd: nlm_release_host() checks for NULL, caller needn't
No need to check for a NULL argument twice. Signed-off-by: Jeff Layton <jlayton@redhat.com> Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
-rw-r--r--fs/lockd/svc4proc.c3
-rw-r--r--fs/lockd/svcproc.c3
2 files changed, 2 insertions, 4 deletions
diff --git a/fs/lockd/svc4proc.c b/fs/lockd/svc4proc.c
index 385437e..006a832 100644
--- a/fs/lockd/svc4proc.c
+++ b/fs/lockd/svc4proc.c
@@ -58,8 +58,7 @@ nlm4svc_retrieve_args(struct svc_rqst *rqstp, struct nlm_args *argp,
return 0;
no_locks:
- if (host)
- nlm_release_host(host);
+ nlm_release_host(host);
if (error)
return error;
return nlm_lck_denied_nolocks;
diff --git a/fs/lockd/svcproc.c b/fs/lockd/svcproc.c
index 88379cc..fce3d70 100644
--- a/fs/lockd/svcproc.c
+++ b/fs/lockd/svcproc.c
@@ -87,8 +87,7 @@ nlmsvc_retrieve_args(struct svc_rqst *rqstp, struct nlm_args *argp,
return 0;
no_locks:
- if (host)
- nlm_release_host(host);
+ nlm_release_host(host);
if (error)
return error;
return nlm_lck_denied_nolocks;
OpenPOWER on IntegriCloud