diff options
author | uqs <uqs@FreeBSD.org> | 2010-12-20 21:12:18 +0000 |
---|---|---|
committer | uqs <uqs@FreeBSD.org> | 2010-12-20 21:12:18 +0000 |
commit | 04f9658f5ccf72df9c57e4e218f11574121fc70f (patch) | |
tree | 975989bb4c9689c6b5a884e4e58d27867689a085 /usr.sbin/rpc.lockd/lockd_lock.c | |
parent | d3eae6a7f5fa0818c6fef993492e3fc27e95a7bb (diff) | |
download | FreeBSD-src-04f9658f5ccf72df9c57e4e218f11574121fc70f.zip FreeBSD-src-04f9658f5ccf72df9c57e4e218f11574121fc70f.tar.gz |
rpc.lockd(8) WARNS cleanup
- Provide function prototype for nlm_syscall
- Don't assign a variable from the stack to a global var[1]
- Remove unused vars
Found by: clang static analyser [1]
Reviewed by: dfr
Diffstat (limited to 'usr.sbin/rpc.lockd/lockd_lock.c')
-rw-r--r-- | usr.sbin/rpc.lockd/lockd_lock.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/usr.sbin/rpc.lockd/lockd_lock.c b/usr.sbin/rpc.lockd/lockd_lock.c index 5d37c32..5900bfa 100644 --- a/usr.sbin/rpc.lockd/lockd_lock.c +++ b/usr.sbin/rpc.lockd/lockd_lock.c @@ -197,6 +197,7 @@ enum hwlock_status lock_hwlock(struct file_lock *fl); enum split_status split_nfslock(const struct file_lock *exist_lock, const struct file_lock *unlock_lock, struct file_lock **left_lock, struct file_lock **right_lock); +int duplicate_block(struct file_lock *fl); void add_blockingfilelock(struct file_lock *fl); enum hwlock_status unlock_hwlock(const struct file_lock *fl); enum hwlock_status test_hwlock(const struct file_lock *fl, @@ -1198,7 +1199,7 @@ test_hwlock(fl, conflicting_fl) int duplicate_block(struct file_lock *fl) { - struct file_lock *ifl,*nfl; + struct file_lock *ifl; int retval = 0; debuglog("Entering duplicate_block"); |