OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_6-14-g986ee1f

Gerrit Code Review gerrit@openafs.org
Thu, 20 Aug 2020 13:34:29 -0400


The following commit has been merged in the openafs-stable-1_8_x branch:
commit 986ee1fd422d1608553af5a6a02bd60349b864ac
Author: Cheyenne Wills <cwills@sinenomine.net>
Date:   Fri Jun 19 08:01:14 2020 -0600

    afs: Avoid panics on failed return from afs_CFileOpen
    
    afs_CFileOpen is a macro that invokes the open "method" of the
    afs_cacheOps structure, and for disk caches the osi_UFSOpen function is
    used.
    
    Currently osi_UFSOpen will panic if there is an error encountered while
    opening a file.
    
    Prepare to handle osi_UFSOpen function returning a NULL instead of
    issuing a panic (future commit).
    
    Update callers of afs_CFileOpen to test for an error and to return an
    error instead of issuing a panic.
    
    While this commit eliminates some panics, it does not address some of the
    more complex cases associated with errors from afs_CFileOpen.
    
    Reviewed-on: https://gerrit.openafs.org/14241
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit d2d27f975df13c3833898611dacff940a5ba3e2a)
    
    Change-Id: Ia30711748b3cffd56eb3120961aa1747dfae0f23
    Reviewed-on: https://gerrit.openafs.org/14286
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/afs/VNOPS/afs_vnop_symlink.c |   11 ++++++++---
 src/afs/afs_buffer.c             |   33 +++++++++++++++++++--------------
 src/afs/afs_dcache.c             |   26 ++++++++++++++++++--------
 src/afs/afs_disconnected.c       |    7 ++++++-
 src/afs/afs_fetchstore.c         |    4 +++-
 src/afs/afs_init.c               |    4 +++-
 src/afs/afs_segments.c           |    7 ++++++-
 7 files changed, 63 insertions(+), 29 deletions(-)

-- 
OpenAFS Master Repository