From 8f03ff3bdd8eb9f4557cdb7054aee9b8ea432160 Mon Sep 17 00:00:00 2001 From: Benjamin Kaduk Date: Sat, 2 Feb 2019 17:10:29 -0600 Subject: [PATCH] dumpscan: appease gcc8 -Wformat-overflow gcc does not benefit from our external knowledge that tm_year is tightly bounded, and thinks it could still be in the range [-2147481748, 2147483647], which would overflow our string buffer. The function in question does not have error handling in place, so rather than adding some or trying to assert the proper bounds, just use a slightly larger buffer for safety. Change-Id: Iafcba5588b805347ddcc0102969bd0e2a3173dd0 Reviewed-on: https://gerrit.openafs.org/13472 Tested-by: BuildBot Reviewed-by: Cheyenne Wills Reviewed-by: Michael Meffie Reviewed-by: Benjamin Kaduk --- src/tools/dumpscan/afsdump_extract.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/tools/dumpscan/afsdump_extract.c b/src/tools/dumpscan/afsdump_extract.c index 98f3a0c271..db0b2fab95 100644 --- a/src/tools/dumpscan/afsdump_extract.c +++ b/src/tools/dumpscan/afsdump_extract.c @@ -237,7 +237,7 @@ static char *month[] = { "Jan", "Feb", "Mar", "Apr", "May", "Jun", static char * datestr(time_t date) { - static char str[13]; + static char str[20]; time_t clock = time(0); struct tm *now, *then; int diff;