]> git.baikalelectronics.ru Git - kernel.git/commit
vfs: Fix EOVERFLOW testing in put_compat_statfs64
authorEric Sandeen <sandeen@redhat.com>
Wed, 2 Oct 2019 21:17:54 +0000 (16:17 -0500)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 3 Oct 2019 21:21:35 +0000 (14:21 -0700)
commit4795aa636df0570ae5e63d76d128cf7479f0d263
tree35fc83c3f92553cce868f9e1f31305a353485c98
parentcce9fb4c8ac4f695dfe0f18de42cc3c82e7db274
vfs: Fix EOVERFLOW testing in put_compat_statfs64

Today, put_compat_statfs64() disallows nearly any field value over
2^32 if f_bsize is only 32 bits, but that makes no sense.
compat_statfs64 is there for the explicit purpose of providing 64-bit
fields for f_files, f_ffree, etc.  And f_bsize is always only 32 bits.

As a result, 32-bit userspace gets -EOVERFLOW for i.e.  large file
counts even with -D_FILE_OFFSET_BITS=64 set.

In reality, only f_bsize and f_frsize can legitimately overflow
(fields like f_type and f_namelen should never be large), so test
only those fields.

This bug was discussed at length some time ago, and this is the proposal
Al suggested at https://lkml.org/lkml/2018/8/6/640.  It seemed to get
dropped amid the discussion of other related changes, but this
part seems obviously correct on its own, so I've picked it up and
sent it, for expediency.

Fixes: 90b610b98aa0 ("vfs: fix put_compat_statfs64() does not handle errors")
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/statfs.c