From d2d3c51a7d76c41b154105c5295eeee172c573ae Mon Sep 17 00:00:00 2001 From: Hans-Werner Paulsen Date: Fri, 9 Nov 2012 13:48:45 +0100 Subject: [PATCH] fileserver: check value of -cb argument The value of the -cb argument for the fileserver was not checked correctly. The fileserver refused to start for any number of callbacks. Change-Id: I25fe9d81ba20d286f36999a554c94fda4ef7fb2f Reviewed-on: http://gerrit.openafs.org/8420 Tested-by: BuildBot Reviewed-by: Jeffrey Altman --- src/viced/viced.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/viced/viced.c b/src/viced/viced.c index 457fdec44..cd5c23075 100644 --- a/src/viced/viced.c +++ b/src/viced/viced.c @@ -1216,9 +1216,11 @@ ParseArgs(int argc, char *argv[]) cmd_OptionAsInt(opts, OPT_buffers, &buffs); if (cmd_OptionAsInt(opts, OPT_callbacks, &numberofcbs) == 0) { - printf("number of cbs %d invalid; " - "must be between 10000 and 2147483647\n", numberofcbs); - return -1; + if ((numberofcbs < 10000) || (numberofcbs > 2147483647)) { + printf("number of cbs %d invalid; " + "must be between 10000 and 2147483647\n", numberofcbs); + return -1; + } } cmd_OptionAsInt(opts, OPT_vcsize, &volcache); -- 2.39.5