OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_3-30-g4d7a397

Gerrit Code Review gerrit@openafs.org
Mon, 10 Jun 2019 08:08:18 -0400


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

    LINUX: Propagate afs_linux_readdir BlobScan errors
    
    In afs_linux_readdir, if we detect an error code from BlobScan,
    currently we 'break' out of the current while() loop. But right after
    this loop, we reset 'code' to 0, ignoring the error we just got from
    BlobScan, and acting like we just reached the end of the directory.
    
    This means that if BlobScan could not process the given directory at
    all, we'll just fail to iterate through some of the entries in the
    given directory, and not report an error.
    
    To fix this, process errors from BlobScan like we do for
    afs_dir_GetVerifiedBlob, and return an error code and log a message
    about the corrupted dir.
    
    Reviewed-on: https://gerrit.openafs.org/13430
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 63f015d05293cd853dbd44e5115e6b378644dfb6)
    
    Change-Id: Ia25bcfdb70cdb1dd1a7ce0efb84ef76beb78b247
    Reviewed-on: https://gerrit.openafs.org/13591
    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/LINUX/osi_vnodeops.c |   14 ++++++++++----
 1 files changed, 10 insertions(+), 4 deletions(-)

-- 
OpenAFS Master Repository