summaryrefslogtreecommitdiffstats
path: root/extras/glusterfs-georep-logrotate
diff options
context:
space:
mode:
authorEmmanuel Dreyfus <manu@netbsd.org>2014-10-15 14:09:16 +0200
committerVijay Bellur <vbellur@redhat.com>2014-10-29 02:26:56 -0700
commit190f43f1818e097a4e4f041c4f56516b067632f0 (patch)
tree97582884b843f553295e4b971bfe6cf75697b418 /extras/glusterfs-georep-logrotate
parentbf6196b876e487c8a37e148b92721f0d85bcfd43 (diff)
Avoid spurious EINVAL in posix_readdir()
On non Linux systems, we check that seekdir() succeeds and we return EINVAL if it does not. We need this to avoid infinite loops if some other component in GlusterFS makes an invalid seekdir() usage. This was introduced in this change: http://review.gluster.org/#/c/8760/ But seekdir() also fails when using the offset returned for the last entry, and this is expected behavior. As a result, the seekdir() test produces a spurious EINVAL when reaching end of directory. That error is not propagated to calling process, but it may harm internal GlusterFS processing. At least it produce a spurious error message in brick's log. We fix the problem by remembering the last entry offset in fd private data. When a new posix_readdir() invocation requests that offset, we avoid returning EINVAL. Backport of I4e67a2ea46538aae63eea663dd4aa33b16ad24c7 BUG: 1138897 Change-Id: I4e98294d157f67ae1a1f0ece1562c77d1219da40 Signed-off-by: Emmanuel Dreyfus <manu@netbsd.org> Reviewed-on: http://review.gluster.org/8933 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'extras/glusterfs-georep-logrotate')
0 files changed, 0 insertions, 0 deletions