From: Garrett Wollman Date: Mon, 16 Jul 2012 02:16:27 +0000 (-0400) Subject: afs_server: remove 3 dead assignments in LoopServers, move live one X-Git-Tag: upstream/1.8.0_pre1^2~2212 X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=4a6a95fdb4ffb8f2681beb5e4e62ddd378baf75d;p=packages%2Fo%2Fopenafs.git afs_server: remove 3 dead assignments in LoopServers, move live one The assignments to conns, rxconns, and conntimer are all dead, so remove them. The assignment to nconns is live, but rather far from the for loop that actually uses it; move it to just before the loop. Change-Id: I8daf642630cc2d468ba7db42de46b69cd88fac28 Reviewed-on: http://gerrit.openafs.org/7777 Reviewed-by: Derrick Brashear Tested-by: BuildBot --- diff --git a/src/afs/afs_server.c b/src/afs/afs_server.c index 98f84010a..92d772a21 100644 --- a/src/afs/afs_server.c +++ b/src/afs/afs_server.c @@ -643,11 +643,6 @@ afs_LoopServers(int adown, struct cell *acellp, int vlalso, if (AFS_IS_DISCONNECTED) return; - conns = (struct afs_conn **)0; - rxconns = (struct rx_connection **) 0; - conntimer = 0; - nconns = 0; - if ((code = afs_InitReq(&treq, afs_osi_credp))) return; ObtainReadLock(&afs_xserver); /* Necessary? */ @@ -681,6 +676,7 @@ afs_LoopServers(int adown, struct cell *acellp, int vlalso, conntimer = afs_osi_Alloc(j * sizeof (afs_int32)); osi_Assert(conntimer != NULL); + nconns = 0; for (i = 0; i < j; i++) { struct rx_connection *rxconn; sa = addrs[i];