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