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