]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
viced: set volume sync data in bulk status rpcs
authorJeffrey Altman <jaltman@secure-endpoints.com>
Thu, 19 Nov 2009 23:11:06 +0000 (18:11 -0500)
committerDerrick Brashear <shadow|account-1000005@unknown>
Thu, 17 Dec 2009 06:19:20 +0000 (22:19 -0800)
The bulkstatus and inlinebulkstatus rpcs have a bug
that prevents the volume sync data from being set.
Currently the data is being set within the for loop
only when i == nfiles.  The conditional of the loop
is i < nfiles so the SetVolumeSync call is never
performed.  This patch changes the test for performing
SetVolumeSync to i == 0.

LICENSE MIT

Change-Id: I17e3980184ec68e38c5186e1c9637ac6e38451fe
Reviewed-on: http://gerrit.openafs.org/856
Reviewed-by: Dan Hyde <drh@umich.edu>
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit 13843d852749ba081804f3af368620db7b4b7d7a)
Reviewed-on: http://gerrit.openafs.org/914

src/viced/afsfileprocs.c

index 5becb0195100d377f32e892a63438b1151ef9d27..92338b4245e910e763b1b6ddfb34a2ab613e7d51 100644 (file)
@@ -2696,7 +2696,7 @@ SRXAFS_BulkStatus(struct rx_call * acall, struct AFSCBFids * Fids,
                              &rights, &anyrights)))
            goto Bad_BulkStatus;
        /* set volume synchronization information, but only once per call */
-       if (i == nfiles)
+       if (i == 0)
            SetVolumeSync(Sync, volptr);
 
        /* Are we allowed to fetch Fid's status? */
@@ -2857,7 +2857,7 @@ SRXAFS_InlineBulkStatus(struct rx_call * acall, struct AFSCBFids * Fids,
        }
 
        /* set volume synchronization information, but only once per call */
-       if (i == nfiles)
+       if (i == 0)
            SetVolumeSync(Sync, volptr);
 
        /* Are we allowed to fetch Fid's status? */