OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_64-9-gf6f9ee5
Gerrit Code Review
gerrit@openafs.org
Thu, 24 Sep 2009 23:05:59 -0700 (PDT)
The following commit has been merged in the master branch:
commit f6f9ee5402f1718f330a00ec89fb34b05c3cd360
Author: Simon Wilkinson <sxw@inf.ed.ac.uk>
Date: Fri Sep 25 00:27:40 2009 +0100
Fix CacheFetchProc in cases where the fileserver hates us
In some situations, the fileserver returns a large negative number
as the length in an FetchData64 call. The old FetchProc code used an int32
to hold this number, and checked length > 0 before attempting to read more
data. The new code uses a uint32, and does while (length), which causes the
cache manage to loop until RX aborts the connection.
This patch restores the old behaviour. length becomes a signed int once more
(and the original 32 bit length from the wire is used, rather than truncating
the 64 bit value), and the conditional checks for > 0.
Reviewed-on: http://gerrit.openafs.org/493
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/afs/afs_fetchstore.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
--
OpenAFS Master Repository