diff options
| author | Jeff Layton <jlayton@redhat.com> | 2017-11-14 20:42:57 +0100 |
|---|---|---|
| committer | Jeff Layton <jlayton@redhat.com> | 2017-11-15 14:08:36 +0100 |
| commit | 4d2dc2cc766c3b51929658cacbc6e34fc8e242fb (patch) | |
| tree | cc8fca80ff984b268f9acedbb582f1f7a410ac69 /arch/sparc | |
| parent | fcntl: don't leak fd reference when fixup_compat_flock fails (diff) | |
| download | linux-4d2dc2cc766c3b51929658cacbc6e34fc8e242fb.tar.xz linux-4d2dc2cc766c3b51929658cacbc6e34fc8e242fb.zip | |
fcntl: don't cap l_start and l_end values for F_GETLK64 in compat syscall
Currently, we're capping the values too low in the F_GETLK64 case. The
fields in that structure are 64-bit values, so we shouldn't need to do
any sort of fixup there.
Make sure we check that assumption at build time in the future however
by ensuring that the sizes we're copying will fit.
With this, we no longer need COMPAT_LOFF_T_MAX either, so remove it.
Fixes: 94073ad77fff2 (fs/locks: don't mess with the address limit in compat_fcntl64)
Reported-by: Vitaly Lipatov <lav@etersoft.ru>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Reviewed-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'arch/sparc')
| -rw-r--r-- | arch/sparc/include/asm/compat.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/sparc/include/asm/compat.h b/arch/sparc/include/asm/compat.h index 977c3f280ba1..fa38c78de0f0 100644 --- a/arch/sparc/include/asm/compat.h +++ b/arch/sparc/include/asm/compat.h @@ -209,7 +209,6 @@ typedef struct compat_siginfo { } compat_siginfo_t; #define COMPAT_OFF_T_MAX 0x7fffffff -#define COMPAT_LOFF_T_MAX 0x7fffffffffffffffL /* * A pointer passed in from user mode. This should not |