diff options
| author | Jeff Layton | 2014-05-09 17:41:54 +0200 |
|---|---|---|
| committer | Jeff Layton | 2014-05-09 17:41:54 +0200 |
| commit | cf01f4eef9fe367ec0d85b38dd7214e29e376cdb (patch) | |
| tree | 2222f99e1cbc1979256996d7fa3bb42196acedfc /fs/ceph | |
| parent | Merge tag 'xfs-for-linus-3.15-rc5' of git://oss.sgi.com/xfs/xfs (diff) | |
| download | kernel-qcow2-linux-cf01f4eef9fe367ec0d85b38dd7214e29e376cdb.tar.gz kernel-qcow2-linux-cf01f4eef9fe367ec0d85b38dd7214e29e376cdb.tar.xz kernel-qcow2-linux-cf01f4eef9fe367ec0d85b38dd7214e29e376cdb.zip | |
locks: only validate the lock vs. f_mode in F_SETLK codepaths
v2: replace missing break in switch statement (as pointed out by Dave
Jones)
commit bce7560d4946 (locks: consolidate checks for compatible
filp->f_mode values in setlk handlers) introduced a regression in the
F_GETLK handler.
flock64_to_posix_lock is a shared codepath between F_GETLK and F_SETLK,
but the f_mode checks should only be applicable to the F_SETLK codepaths
according to POSIX.
Instead of just reverting the patch, add a new function to do this
checking and have the F_SETLK handlers call it.
Cc: Dave Jones <davej@redhat.com>
Reported-and-Tested-by: Reuben Farrelly <reuben@reub.net>
Signed-off-by: Jeff Layton <jlayton@poochiereds.net>
Diffstat (limited to 'fs/ceph')
0 files changed, 0 insertions, 0 deletions