Re: [PATCH] vfs: in iomap seek_{hole,data}, return -ENXIO for negative offsets

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Jul 12, 2017, at 10:36 AM, Darrick J. Wong <darrick.wong@xxxxxxxxxx> wrote:
> 
> In the iomap implementations of SEEK_HOLE and SEEK_DATA, make sure we
> return -ENXIO for negative offsets instead of badgering the iomap
> provider with garbage requests.

Hmm, wouldn't it be useful to be able to seek N bytes before the hole or data?
It would make more sense to verify the result after finding the hole or data
and adding the relative offset. It should only be an error if the resulting
offset is before the start or after the actual file end.  Of course, if
the passed offset > size from the start then there is no way it can get better
and that would be grounds for returning early.

Cheers, Andreas

> Inspired-by: Mateusz S <muttdini@xxxxxxxxx>
> Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> ---
> fs/iomap.c |    8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/fs/iomap.c b/fs/iomap.c
> index 432eed8..16f5c074 100644
> --- a/fs/iomap.c
> +++ b/fs/iomap.c
> @@ -610,8 +610,8 @@ iomap_seek_hole(struct inode *inode, loff_t offset, const struct iomap_ops *ops)
> 	loff_t length = size - offset;
> 	loff_t ret;
> 
> -	/* Nothing to be found beyond the end of the file. */
> -	if (offset >= size)
> +	/* Nothing to be found before or beyond the end of the file. */
> +	if (offset < 0 || offset >= size)
> 		return -ENXIO;
> 
> 	while (length > 0) {
> @@ -656,8 +656,8 @@ iomap_seek_data(struct inode *inode, loff_t offset, const struct iomap_ops *ops)
> 	loff_t length = size - offset;
> 	loff_t ret;
> 
> -	/* Nothing to be found beyond the end of the file. */
> -	if (offset >= size)
> +	/* Nothing to be found before or beyond the end of the file. */
> +	if (offset < 0 || offset >= size)
> 		return -ENXIO;
> 
> 	while (length > 0) {


Cheers, Andreas





Attachment: signature.asc
Description: Message signed with OpenPGP


[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux