Fix readdir() bug when a non-zero offset is specified in filler (#269)
authorRostislav <rostislav@users.noreply.github.com>
Sat, 21 Jul 2018 09:57:09 +0000 (12:57 +0300)
committerNikolaus Rath <Nikolaus@rath.org>
Sat, 21 Jul 2018 09:57:09 +0000 (10:57 +0100)
commit5f125c5e6be24c8d216a4d3c623dc73d742c8c86
treedc98c1f6db3695d6cf46276f0674bff243fd646e
parent795ad5d77434f3502e63a70c8a3fda94fa347e3d
Fix readdir() bug when a non-zero offset is specified in filler (#269)

The bug occurs when a filesystem client reads a directory until the end,
seeks using seekdir() to some valid non-zero position and calls
readdir(). A valid 'struct dirent *' is expected, but NULL is returned
instead. Pseudocode demonstrating the bug:

DIR *dp = opendir("some_dir");
struct dirent *de = readdir(dp);

/* Get offset of the second entry */
long offset = telldir(dp);

/* Read directory until the end */
while (de)
de = readdir(de);

seekdir(dp, offset);
de = readdir(dp);
/* de must contain the second entry, but NULL is returned instead */

The reason of the bug is that when the end of directory is reached, the
kernel calls FUSE_READDIR op with an offset at the end of directory, so
the filesystem's .readdir callback never calls the filler function, and
we end up with dh->filled set to 1. After seekdir(), FUSE_READDIR is
called again with a new offset, but this time the filesystem's .readdir
callback is never called, and an empty reply is returned.

Fix by setting dh->filled to 1 only when zero offsets are given to
filler function.
ChangeLog.rst
lib/fuse.c
test/test_syscalls.c