ext4_seek_hole-return-ENXIO-for-negative-offsets

   1 ext4: in ext4_seek_{hole,data}, return -ENXIO for negative offsets
   2
   3 From: "Darrick J. Wong" <darrick.wong@oracle.com>
   4
   5 In the ext4 implementations of SEEK_HOLE and SEEK_DATA, make sure we
   6 return -ENXIO for negative offsets instead of banging around inside
   7 the extent code and returning -EFSCORRUPTED.
   8
   9 Reported-by: Mateusz S <muttdini@gmail.com>
  10 Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
  11 Cc: stable@vger.kernel.org # 4.6
  12 ---
  13  fs/ext4/file.c |    4 ++--
  14  1 file changed, 2 insertions(+), 2 deletions(-)
  15
  16 diff --git a/fs/ext4/file.c b/fs/ext4/file.c
  17 index 02ce7e7..923e765 100644
  18 --- a/fs/ext4/file.c
  19 +++ b/fs/ext4/file.c
  20 @@ -576,7 +576,7 @@ static loff_t ext4_seek_data(struct file *file, loff_t offset, loff_t maxsize)
  21         inode_lock(inode);
  22
  23         isize = i_size_read(inode);
  24 -       if (offset >= isize) {
  25 +       if (offset < 0 || offset >= isize) {
  26                 inode_unlock(inode);
  27                 return -ENXIO;
  28         }
  29 @@ -639,7 +639,7 @@ static loff_t ext4_seek_hole(struct file *file, loff_t offset, loff_t maxsize)
  30         inode_lock(inode);
  31
  32         isize = i_size_read(inode);
  33 -       if (offset >= isize) {
  34 +       if (offset < 0 || offset >= isize) {
  35                 inode_unlock(inode);
  36                 return -ENXIO;
  37         }
  38