diff options
author | J. Bruce Fields <bfields@redhat.com> | 2022-01-18 17:00:51 -0500 |
---|---|---|
committer | Chuck Lever <chuck.lever@oracle.com> | 2022-01-27 10:45:25 -0500 |
commit | d19a7af73b5ecaac8168712d18be72b9db166768 (patch) | |
tree | b04743e6bf80e7af7c5bbde22c7137b3b1a75096 /fs/fs_struct.c | |
parent | 6e7f90d163afa8fc2efd6ae318e7c20156a5621f (diff) | |
download | linux-stable-d19a7af73b5ecaac8168712d18be72b9db166768.tar.gz linux-stable-d19a7af73b5ecaac8168712d18be72b9db166768.tar.bz2 linux-stable-d19a7af73b5ecaac8168712d18be72b9db166768.zip |
lockd: fix failure to cleanup client locks
In my testing, we're sometimes hitting the request->fl_flags & FL_EXISTS
case in posix_lock_inode, presumably just by random luck since we're not
actually initializing fl_flags here.
This probably didn't matter before commit 7f024fcd5c97 ("Keep read and
write fds with each nlm_file") since we wouldn't previously unlock
unless we knew there were locks.
But now it causes lockd to give up on removing more locks.
We could just initialize fl_flags, but really it seems dubious to be
calling vfs_lock_file with random values in some of the fields.
Fixes: 7f024fcd5c97 ("Keep read and write fds with each nlm_file")
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
[ cel: fixed checkpatch.pl nit ]
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'fs/fs_struct.c')
0 files changed, 0 insertions, 0 deletions