Commit da565e79 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Theodore Ts'o
Browse files

ext4: remove the call to fiemap_check_flags in ext4_fiemap 



iomap_fiemap already calls fiemap_check_flags first thing, so this
additional check is redundant.

Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarRitesh Harjani <riteshh@linux.ibm.com>
Reviewed-by: default avatarJan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20200523073016.2944131-3-hch@lst.de


Signed-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
parent 03a5ed24

fs/ext4/extents.c

+0 −3
Original line number Diff line number Diff line
@@ -4896,9 +4896,6 @@ int ext4_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo, 
		fieinfo->fi_flags &= ~FIEMAP_FLAG_CACHE;

	}



	if (fiemap_check_flags(fieinfo, FIEMAP_FLAG_SYNC | FIEMAP_FLAG_XATTR))

		return -EBADR;



	/*

	 * For bitmap files the maximum size limit could be smaller than

	 * s_maxbytes, so check len here manually instead of just relying on the