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