From 71e05b1dcf8a2ae47d59af8ab6e976a900ab9dc2 Mon Sep 17 00:00:00 2001 From: Baptiste Jonglez Date: Fri, 6 Jan 2012 23:49:51 +0100 Subject: src/apoint.c: Fix a length bug introduced by new duration format Commit 4ff3bb9d introduced a new format for parsing durations, thus allowing a larger string to be retrieved from user input. However, the string used is still declared with the old length, leading to crashes when compiling with stack-smashing protection features (the default on Archlinux). Inputting a duration string of more than 8 characters (such as "+1d11h11m") would crash calcurse with a *** stack smashing detected *** message. Using a larger string from the start fixes the bug. Signed-off-by: Baptiste Jonglez Signed-off-by: Lukas Fleischer --- src/apoint.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/apoint.c b/src/apoint.c index 9e0d1b9..aa2a516 100644 --- a/src/apoint.c +++ b/src/apoint.c @@ -166,7 +166,7 @@ apoint_add (void) _("You entered an invalid end time, should be [hh:mm], [+hh:mm] or [+mm]"); char *enter_str = _("Press [Enter] to continue"); int Id = 1; - char item_time[LTIME] = ""; + char item_time[LDUR] = ""; char item_mesg[BUFSIZ] = ""; long apoint_start; unsigned heures, minutes; -- cgit v1.2.3-70-g09d2