diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2009-05-02 19:05:37 -0400 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2009-05-02 19:05:37 -0400 |
| commit | eefd7f03b86b8a319890e7fac5a6fcc7f8694b76 (patch) | |
| tree | d8dd02626d522926b828042f87c0850d42f86d2e /fs/ext4 | |
| parent | c9877b205f6ce7943bb95281342f4001cc1c00ec (diff) | |
| download | op-kernel-dev-eefd7f03b86b8a319890e7fac5a6fcc7f8694b76.zip op-kernel-dev-eefd7f03b86b8a319890e7fac5a6fcc7f8694b76.tar.gz | |
ext4: fix the length returned by fiemap for an unallocated extent
If the file's blocks have not yet been allocated because of delayed
allocation, the length of the extent returned by fiemap is incorrect.
This commit fixes this bug.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
| -rw-r--r-- | fs/ext4/extents.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index 5f72952..4fec6b7 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -3244,8 +3244,15 @@ static int ext4_ext_fiemap_cb(struct inode *inode, struct ext4_ext_path *path, * XXX this might miss a single-block extent at EXT_MAX_BLOCK */ if (ext4_ext_next_allocated_block(path) == EXT_MAX_BLOCK || - newex->ec_block + newex->ec_len - 1 == EXT_MAX_BLOCK) + newex->ec_block + newex->ec_len - 1 == EXT_MAX_BLOCK) { + loff_t size = i_size_read(inode); + loff_t bs = EXT4_BLOCK_SIZE(inode->i_sb); + flags |= FIEMAP_EXTENT_LAST; + if ((flags & FIEMAP_EXTENT_DELALLOC) && + logical+length > size) + length = (size - logical + bs - 1) & ~(bs-1); + } error = fiemap_fill_next_extent(fieinfo, logical, physical, length, flags); |
