The following commit has been merged in the openafs-stable-1_8_x branch:
commit 986ee1fd422d1608553af5a6a02bd60349b864ac
Author: Cheyenne Wills <[email protected]>
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 <[email protected]>
    Reviewed-by: Mark Vitale <[email protected]>
    Tested-by: BuildBot <[email protected]>
    Reviewed-by: Benjamin Kaduk <[email protected]>
    (cherry picked from commit d2d27f975df13c3833898611dacff940a5ba3e2a)
    
    Change-Id: Ia30711748b3cffd56eb3120961aa1747dfae0f23
    Reviewed-on: https://gerrit.openafs.org/14286
    Tested-by: BuildBot <[email protected]>
    Reviewed-by: Andrew Deason <[email protected]>
    Reviewed-by: Cheyenne Wills <[email protected]>
    Reviewed-by: Stephan Wiesand <[email protected]>

 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
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs

Reply via email to