From: Derrick Brashear <shadow@dementix.org>
Date: Fri, 29 Jun 2012 14:04:10 +0000 (-0400)
Subject: viced: null-terminate server list for ubik
X-Git-Tag: upstream/1.8.0_pre1^2~2305
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=0c6a2a7b359b7b0432c94287485ed1ed26d5f253;p=packages%2Fo%2Fopenafs.git

viced: null-terminate server list for ubik

when we set up the vlserver connection, we could potentially use
uninitialized memory as ubik connections. don't.

Change-Id: Idefa71874c7402c927fbf148f994f9422183208a
Reviewed-on: http://gerrit.openafs.org/7610
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
Tested-by: Derrick Brashear <shadow@dementix.org>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
---

diff --git a/src/viced/viced.c b/src/viced/viced.c
index 982aad2c2..c448d8002 100644
--- a/src/viced/viced.c
+++ b/src/viced/viced.c
@@ -1494,6 +1494,7 @@ vl_Initialize(struct afsconf_dir *dir)
     struct rx_securityClass *sc;
     struct rx_connection *serverconns[MAXSERVERS];
 
+    memset(serverconns, 0, sizeof(serverconns));
     code = afsconf_ClientAuth(dir, &sc, &scIndex);
     if (code) {
 	ViceLog(0, ("Could not get security object for localAuth\n"));