Re: Invalid DIR * usage in quota xlator

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

 



On Wed, Oct 15, 2014 at 07:42:25PM +0200, Emmanuel Dreyfus wrote:
> J. Bruce Fields <bfields@xxxxxxxxxxxx> wrote:
> 
> > You're assuming no filesystem uses -1 as a cookie?
> > Might be true, but it's taking a small chance.
> 
> Latest version does not work like this anymore: when EOF is reached,
> current offset is recorded in pfd->dir_eof, and we fire EINVAL if
> seekdir() failed to set the offset and the offset is not  pfd->dir_eof:
> 
>                 if (telldir(dir) != (long)off && off != pfd->dir_eof)

But actually, I was wrong: telldir returns -1 for error or offset on
success, so -1 is reserved.

--b.
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://supercolony.gluster.org/mailman/listinfo/gluster-devel




[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux