OpenAFS Master Repository branch, master, updated. BP-openafs-stable-1_8_x-479-g360b9d5
Gerrit Code Review
gerrit@openafs.org
Fri, 8 Nov 2019 03:01:03 -0500
The following commit has been merged in the master branch:
commit 360b9d5d71fb1de142ae4efd4660732476855a3f
Author: Andrew Deason <adeason@sinenomine.net>
Date: Mon Nov 4 20:03:43 2019 -0600
afs: Avoid -1 error for vreadUIO/vwriteUIO
Commit c6b61a45 (afs: Verify osi_UFSOpen worked) added various checks
to return an error if a given osi_UFSOpen failed. However, two of
these checks (in afs_UFSReadUIO and afs_UFSWriteUIO) result in us
returning -1 on error, in functions that otherwise return errno codes
(e.g. ENOSPC). An error code of -1 might get interpreted as
RX_CALL_DEAD, which would be rather confusing, so use EIO as a generic
error instead.
Change-Id: I23b9a73b82d999d8ee4670b5e7ec39b9d820fb0f
Reviewed-on: https://gerrit.openafs.org/13931
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
src/afs/VNOPS/afs_vnop_read.c | 2 +-
src/afs/VNOPS/afs_vnop_write.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
OpenAFS Master Repository