This is a note to let you know that I've just added the patch titled lockd: fix file selection in nlmsvc_cancel_blocked to the 6.1-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: lockd-fix-file-selection-in-nlmsvc_cancel_blocked.patch and it can be found in the queue-6.1 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From fdf4fd6f05ae4659354de48f42dcee40eb4405dd Mon Sep 17 00:00:00 2001 From: Jeff Layton <jlayton@xxxxxxxxxx> Date: Fri, 11 Nov 2022 14:36:38 -0500 Subject: lockd: fix file selection in nlmsvc_cancel_blocked From: Jeff Layton <jlayton@xxxxxxxxxx> [ Upstream commit 9f27783b4dd235ef3c8dbf69fc6322777450323c ] We currently do a lock_to_openmode call based on the arguments from the NLM_UNLOCK call, but that will always set the fl_type of the lock to F_UNLCK, and the O_RDONLY descriptor is always chosen. Fix it to use the file_lock from the block instead. Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- fs/lockd/svclock.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) --- a/fs/lockd/svclock.c +++ b/fs/lockd/svclock.c @@ -699,9 +699,10 @@ nlmsvc_cancel_blocked(struct net *net, s block = nlmsvc_lookup_block(file, lock); mutex_unlock(&file->f_mutex); if (block != NULL) { - mode = lock_to_openmode(&lock->fl); - vfs_cancel_lock(block->b_file->f_file[mode], - &block->b_call->a_args.lock.fl); + struct file_lock *fl = &block->b_call->a_args.lock.fl; + + mode = lock_to_openmode(fl); + vfs_cancel_lock(block->b_file->f_file[mode], fl); status = nlmsvc_unlink_block(block); nlmsvc_release_block(block); } Patches currently in stable-queue which might be from jlayton@xxxxxxxxxx are queue-6.1/nfsd-refactor-find_file.patch queue-6.1/lockd-use-locks_inode_context-helper.patch queue-6.1/nfsd-remove-redundant-assignment-to-variable-host_err.patch queue-6.1/nfsd-ignore-requests-to-disable-unsupported-versions.patch queue-6.1/nfsd-fix-licensing-header-in-filecache.c.patch queue-6.1/nfsd-trace-stateids-returned-via-delegreturn.patch queue-6.1/lockd-ensure-we-use-the-correct-file-descriptor-when-unlocking.patch queue-6.1/nfsd-update-file_hashtbl-helpers.patch queue-6.1/nfsd-clean-up-nfs4_preprocess_stateid_op-call-sites.patch queue-6.1/nfsd-don-t-destroy-global-nfs4_file-table-in-per-net-shutdown.patch queue-6.1/nfsd-use-only-rq_dropme-to-signal-the-need-to-drop-a-reply.patch queue-6.1/nfsd-use-locks_inode_context-helper.patch queue-6.1/nfsd-use-struct_size-helper-in-alloc_session.patch queue-6.1/nfsd-fix-up-the-filecache-laundrette-scheduling.patch queue-6.1/nfsd-move-nfserrno-to-vfs.c.patch queue-6.1/nfsd-use-const-pointers-as-parameters-to-fh_-helpers.patch queue-6.1/nfsd-use-rhashtable-for-managing-nfs4_file-objects.patch queue-6.1/nfsd-clean-up-nfsd4_init_file.patch queue-6.1/nfsd-add-a-nfsd4_file_hash_remove-helper.patch queue-6.1/nfsd-trace-delegation-revocations.patch queue-6.1/nfsd-allow-disabling-nfsv2-at-compile-time.patch queue-6.1/nfsd-flesh-out-a-documenting-comment-for-filecache.c.patch queue-6.1/nfsd-replace-delayed_work-with-work_struct-for-nfsd_client_shrinker.patch queue-6.1/lockd-set-missing-fl_flags-field-when-retrieving-args.patch queue-6.1/filelock-add-a-new-locks_inode_context-accessor-function.patch queue-6.1/lockd-fix-file-selection-in-nlmsvc_cancel_blocked.patch queue-6.1/nfsd-avoid-clashing-function-prototypes.patch