From e75e4b2ed8eddf43d4752dae2dc571fc02e0137d Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Fri, 11 Jul 2014 13:19:36 +0200 Subject: [PATCH] smbd: Rename lck1->lock in brl_conflict_other lck1 and lck2 are treated differently. They should carry more descriptive names. Signed-off-by: Volker Lendecke Reviewed-by: Stefan Metzmacher --- source3/locking/brlock.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/source3/locking/brlock.c b/source3/locking/brlock.c index 51a0fd93578..56c700e5ece 100644 --- a/source3/locking/brlock.c +++ b/source3/locking/brlock.c @@ -227,18 +227,18 @@ static bool brl_conflict1(const struct lock_struct *lck1, This is never used in the POSIX lock case. ****************************************************************************/ -static bool brl_conflict_other(const struct lock_struct *lck1, const struct lock_struct *lck2) +static bool brl_conflict_other(const struct lock_struct *lock, const struct lock_struct *lck2) { - if (IS_PENDING_LOCK(lck1->lock_type) || IS_PENDING_LOCK(lck2->lock_type)) + if (IS_PENDING_LOCK(lock->lock_type) || IS_PENDING_LOCK(lck2->lock_type)) return False; - if (lck1->lock_type == READ_LOCK && lck2->lock_type == READ_LOCK) + if (lock->lock_type == READ_LOCK && lck2->lock_type == READ_LOCK) return False; /* POSIX flavour locks never conflict here - this is only called in the read/write path. */ - if (lck1->lock_flav == POSIX_LOCK && lck2->lock_flav == POSIX_LOCK) + if (lock->lock_flav == POSIX_LOCK && lck2->lock_flav == POSIX_LOCK) return False; /* @@ -246,13 +246,13 @@ static bool brl_conflict_other(const struct lock_struct *lck1, const struct lock * if the context is the same. JRA. See LOCKTEST7 in smbtorture. */ - if (!(lck2->lock_type == WRITE_LOCK && lck1->lock_type == READ_LOCK)) { - if (brl_same_context(&lck1->context, &lck2->context) && - lck1->fnum == lck2->fnum) + if (!(lck2->lock_type == WRITE_LOCK && lock->lock_type == READ_LOCK)) { + if (brl_same_context(&lock->context, &lck2->context) && + lock->fnum == lck2->fnum) return False; } - return brl_overlap(lck1, lck2); + return brl_overlap(lock, lck2); } /**************************************************************************** -- 2.11.4.GIT