From 5cefa0247fe4f22f769e366a207fb177ee8fb94f Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Fri, 22 Feb 2013 09:51:24 +0000 Subject: [PATCH] kauth: Don't ignore GetExtendedCellInfo failures Don't ignore a failure from afsconf_GetExtendedCellInfo, as it will leave buffers that we later read uninitialised. Caught by clang-analyzer Change-Id: Ia0c2675a3083385d011fd340f36bf13373f89feb Reviewed-on: http://gerrit.openafs.org/9180 Tested-by: BuildBot Reviewed-by: Derrick Brashear Reviewed-by: Jeffrey Altman --- src/kauth/kaserver.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/kauth/kaserver.c b/src/kauth/kaserver.c index 108bfa98d..2fca96a4d 100644 --- a/src/kauth/kaserver.c +++ b/src/kauth/kaserver.c @@ -328,9 +328,13 @@ main(int argc, char *argv[]) "Migrating to a Kerberos 5 KDC is advised. " "http://www.openafs.org/no-more-des.html\n")); - code = - afsconf_GetExtendedCellInfo(KA_conf, cell, AFSCONF_KAUTHSERVICE, - &cellinfo, clones); + code = afsconf_GetExtendedCellInfo(KA_conf, cell, AFSCONF_KAUTHSERVICE, + &cellinfo, clones); + if (code) { + afs_com_err(whoami, code, "Couldn't read cell configuration"); + exit(1); + } + if (servers) { if ((code = ubik_ParseServerList(argc, argv, &myHost, serverList))) { afs_com_err(whoami, code, "Couldn't parse server list"); -- 2.39.5