From 13843d852749ba081804f3af368620db7b4b7d7a Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Thu, 19 Nov 2009 18:11:06 -0500 Subject: [PATCH] viced: set volume sync data in bulk status rpcs 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 Tested-by: Derrick Brashear Reviewed-by: Derrick Brashear --- src/viced/afsfileprocs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/viced/afsfileprocs.c b/src/viced/afsfileprocs.c index 3cad9c7092..fb581a5a1c 100644 --- a/src/viced/afsfileprocs.c +++ b/src/viced/afsfileprocs.c @@ -2641,7 +2641,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? */ @@ -2802,7 +2802,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? */