OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_3-29-gb924e78

[Gerrit Code Review]

The following commit has been merged in the openafs-stable-1_8_x branch:
commit b924e7827dbca97accb6a90da24264858e9a2964
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Wed Jan 16 23:46:34 2019 -0600

    afs: Throw EIO in DRead on empty dir blob
    
    DRead currently returns ENOENT if we try to read a page beyond the end
    of the given dir blob. We do this to indicate we've hit EOF, but we do
    this even if the dir blob is completely empty (which is not a valid
    dir blob).
    
    If a dir blob in the cache is truncated due to cache corruption
    issues, that means we'll indicate a normal EOF condition in that
    directory for most code paths. If someone is trying to list the
    directory's entries, for instance, we'll just return that there are no
    entries in the dir, even though the dir itself is just invalid.
    
    To avoid this for at least some cases, return an EIO error instead if
    the dir blob is completely empty.
    
    Reviewed-on: https://gerrit.openafs.org/13429
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 86d04ea70fd2e99606b1d1b5b68d980d92e7a3cd)
    
    Change-Id: I067aae1f949051169225a3cc0bdba35ad76a4ec2
    Reviewed-on: https://gerrit.openafs.org/13590
    Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/afs/afs_buffer.c |   12 ++++++++++--
 1 files changed, 10 insertions(+), 2 deletions(-)

-- 
OpenAFS Master Repository