[PATCH] initialize buf2 properly

Julius Plenz plenz at cis.fu-berlin.de
Wed Apr 6 13:55:19 CEST 2011


I don't see why you would want buf2 to be static in nature. If you have
a file that contains a single newline, for example, parse_date() returns
-1. buf2 still contains the previous date's file.

Consequently, if you call read_agefile() several times in a row, the
stack might just contain the previous date still. Thus, there's a need
to initialize it properly.

Signed-off-by: Julius Plenz <plenz at cis.fu-berlin.de>
---
 ui-repolist.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/ui-repolist.c b/ui-repolist.c
index 2c98668..4431145 100644
--- a/ui-repolist.c
+++ b/ui-repolist.c
@@ -15,11 +15,12 @@ time_t read_agefile(char *path)
 	time_t result;
 	size_t size;
 	char *buf;
-	static char buf2[64];
+	char buf2[64];
 
 	if (readfile(path, &buf, &size))
 		return -1;
 
+	memset(buf2, 0, 64);
 	if (parse_date(buf, buf2, sizeof(buf2)))
 		result = strtoul(buf2, NULL, 10);
 	else
-- 
1.7.3.1





More information about the CGit mailing list