summaryrefslogtreecommitdiff
path: root/source4/ntvfs
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2007-05-14 17:56:00 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 14:52:25 -0500
commit5eb7420f119df890e2e4b7acd904dc2da70793a6 (patch)
tree6ff37658214b5454182d3ebb547ebacf3f7adf1e /source4/ntvfs
parent0543c06d2598ccd476bcb78e7701e50e6a85e51f (diff)
downloadsamba-5eb7420f119df890e2e4b7acd904dc2da70793a6.tar.gz
r22864: in SMB2 mode we need to cancel an existing lock with a conflicting lock
if they're on the same handle and the same range metze (This used to be commit 36384bb4a071dcc366ae1039bd0579f08b84e238)
Diffstat (limited to 'source4/ntvfs')
-rw-r--r--source4/ntvfs/posix/pvfs_lock.c13
1 files changed, 11 insertions, 2 deletions
diff --git a/source4/ntvfs/posix/pvfs_lock.c b/source4/ntvfs/posix/pvfs_lock.c
index 4527cd3ed2..9b5823e69e 100644
--- a/source4/ntvfs/posix/pvfs_lock.c
+++ b/source4/ntvfs/posix/pvfs_lock.c
@@ -69,8 +69,13 @@ static void pvfs_lock_async_failed(struct pvfs_state *pvfs,
int i,
NTSTATUS status)
{
+ /* in SMB2 mode we also try to unlock failing lock */
+ if (req->ctx->protocol != PROTOCOL_SMB2) {
+ i--;
+ }
+
/* undo the locks we just did */
- for (i=i-1;i>=0;i--) {
+ for (;i>=0;i--) {
brl_unlock(pvfs->brl_context,
f->brl_handle,
locks[i].pid,
@@ -377,8 +382,12 @@ NTSTATUS pvfs_lock(struct ntvfs_module_context *ntvfs,
DLIST_ADD(f->pending_list, pending);
return NT_STATUS_OK;
}
+ /* in SMB2 mode we also try to unlock failing lock */
+ if (req->ctx->protocol != PROTOCOL_SMB2) {
+ i--;
+ }
/* undo the locks we just did */
- for (i=i-1;i>=0;i--) {
+ for (;i>=0;i--) {
brl_unlock(pvfs->brl_context,
f->brl_handle,
locks[i].pid,