Unverified Commit 1f7c7602 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!10394 filelock: Fix fcntl/close race recovery compat path 

Merge Pull Request from: @ci-robot 
 
PR sync from: Yifan Qiao <qiaoyifan4@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/B7E3JQINKGS2SQ4H3JM76KPB2ZYMDBL3/ 
 
https://gitee.com/src-openeuler/kernel/issues/IAGEF4 
 
Link:https://gitee.com/openeuler/kernel/pulls/10394

 

Reviewed-by: default avatarZhang Peng <zhangpeng362@huawei.com>
Signed-off-by: default avatarZhang Peng <zhangpeng362@huawei.com>
parents 37156116 a34f4c16

fs/locks.c

+4 −5
Original line number Diff line number Diff line
@@ -2504,8 +2504,9 @@ int fcntl_setlk64(unsigned int fd, struct file *filp, unsigned int cmd, 
	error = do_lock_file_wait(filp, cmd, file_lock);



	/*

	 * Attempt to detect a close/fcntl race and recover by releasing the

	 * lock that was just acquired. There is no need to do that when we're

	 * Detect close/fcntl races and recover by zapping all POSIX locks

	 * associated with this file and our files_struct, just like on

	 * filp_flush(). There is no need to do that when we're

	 * unlocking though, or for OFD locks.

	 */

	if (!error && file_lock->fl_type != F_UNLCK &&
@@ -2520,9 +2521,7 @@ int fcntl_setlk64(unsigned int fd, struct file *filp, unsigned int cmd, 
		f = files_lookup_fd_locked(files, fd);

		spin_unlock(&files->file_lock);

		if (f != filp) {

			file_lock->fl_type = F_UNLCK;

			error = do_lock_file_wait(filp, cmd, file_lock);

			WARN_ON_ONCE(error);

			locks_remove_posix(filp, files);

			error = -EBADF;

		}

	}