From 252a5b65c30636bf5f62a49a79e5777004ecd17d Mon Sep 17 00:00:00 2001 From: Ben Kaduk Date: Tue, 19 Feb 2013 13:06:36 -0500 Subject: [PATCH] butc: do not use stack garbage for secobj The butc only ever handles connections over rxnull anyway, so just specify an array of security objects of length one. Change-Id: Ia073157488ac6b887f972448074ad1615422e26e Reviewed-on: http://gerrit.openafs.org/9140 Tested-by: BuildBot Reviewed-by: Simon Wilkinson Reviewed-by: Derrick Brashear --- src/butc/tcmain.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/butc/tcmain.c b/src/butc/tcmain.c index ada509906..885d44c4e 100644 --- a/src/butc/tcmain.c +++ b/src/butc/tcmain.c @@ -835,7 +835,7 @@ static int WorkerBee(struct cmd_syndesc *as, void *arock) { afs_int32 code; - struct rx_securityClass *(securityObjects[3]); + struct rx_securityClass *(securityObjects[1]); struct rx_service *service; time_t tokenExpires; char cellName[64]; @@ -1083,14 +1083,13 @@ WorkerBee(struct cmd_syndesc *as, void *arock) */ securityObjects[0] = rxnull_NewServerSecurityObject(); - securityObjects[1] = (struct rx_securityClass *)0; /* don't bother with rxvab */ if (!securityObjects[0]) { TLog(0, "rxnull_NewServerSecurityObject"); exit(1); } service = - rx_NewServiceHost(host, 0, 1, "BUTC", securityObjects, 3, TC_ExecuteRequest); + rx_NewServiceHost(host, 0, 1, "BUTC", securityObjects, 1, TC_ExecuteRequest); if (!service) { TLog(0, "rx_NewService"); exit(1); -- 2.39.5