The following commit has been merged in the openafs-stable-1_6_x branch:
commit 9feb69bb4c8417f0a8006fd43f6bc15e0b60cefd
Author: Simon Wilkinson <[email protected]>
Date: Fri Mar 30 19:14:38 2012 +0100
libadmin: read returns an ssize_t, not a size_t
size_t is unsigned, and therefore can never be less than 0. Using it as
a return code from read() means that we never catch read errors. read()
is defined as returning ssize_t, so just use this to capture its return
code.
Caught by clang's new error messages
Reviewed-on: http://gerrit.openafs.org/7075
Reviewed-by: Derrick Brashear <[email protected]>
Tested-by: Derrick Brashear <[email protected]>
(cherry picked from commit b5f75c122875b4212eb1a885d09bfdd63f537a19)
Change-Id: Ief5b6cd7751cf68c4254cd113da734c864d53d5d
Reviewed-on: http://gerrit.openafs.org/10741
Tested-by: BuildBot <[email protected]>
Reviewed-by: Derrick Brashear <[email protected]>
Reviewed-by: Andrew Deason <[email protected]>
Reviewed-by: Stephan Wiesand <[email protected]>
src/libadmin/bos/afs_bosAdmin.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
--
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs
