From 96385ddaee0cc31c0ea7e5e9a900a7e58891f82c 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. Reviewed-on: http://gerrit.openafs.org/9140 Tested-by: BuildBot Reviewed-by: Simon Wilkinson Reviewed-by: Derrick Brashear (cherry picked from commit 252a5b65c30636bf5f62a49a79e5777004ecd17d) Change-Id: I0b5c112f56553b7ee0ea4e40a9c65f2fe45f8297 Reviewed-on: http://gerrit.openafs.org/9404 Reviewed-by: Jeffrey Altman Tested-by: BuildBot Reviewed-by: Derrick Brashear Reviewed-by: Andrew Deason Reviewed-by: Paul Smeddle Reviewed-by: Stephan Wiesand --- 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 bdfb98039..35b2bc49a 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]; @@ -1084,14 +1084,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