OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-4730-geca8674

Gerrit Code Review gerrit@openafs.org
Wed, 11 Nov 2015 10:27:19 -0500


The following commit has been merged in the master branch:
commit eca86749d8f158d27d131ecaafe6be282703535e
Author: Michael Meffie <mmeffie@sinenomine.net>
Date:   Mon Oct 12 22:16:54 2015 -0400

    afs: fix for return an error from afs_readdir when out of buffers
    
    Commit 9b0d5f274fe79ccc5dd0e4bba86b3f52b27d3586 added a return code to
    BlobScan to allow afs_readdir to return an error when afs_newslot failed
    to allocate a buffer.  Unfortunately, that change introduced a false
    EIO error.
    
    Originally, BlobScan would return a blob number of 0 to indicate the end
    of the file has been reached while traversing the directory blobs.
    Restore that behavior by changing the cache manager's DRead function to
    return ENOENT instead of the generic EIO error to indicate the page to
    be read is out of bounds, and change BlobScan to return a blob of zero
    to indicate to callers the last blob has been reached.  All callers
    already check for a blob number of zero, which is out of range.
    
    Change-Id: I5baae8e5377dd49dcca6765b7a4ddc89cca70738
    Reviewed-on: http://gerrit.openafs.org/12058
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Tested-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

 src/afs/VNOPS/afs_vnop_readdir.c |    8 +++++++-
 src/afs/afs_buffer.c             |    2 +-
 2 files changed, 8 insertions(+), 2 deletions(-)

-- 
OpenAFS Master Repository