fyb/calcurse-edge
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use strncpy() properly in general_option_edit()

Browse Source 
Always use strncpy() to copy strings between fixed-size buffers and pass
the buffer size as maximal length parameter.

Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>
This commit is contained in:
Lukas Fleischer 2017-09-03 09:28:27 +02:00
parent 578091f051
commit 6521d8cc0a
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/custom.c
View File

@ -739,11 +739,11 @@ static void general_option_edit(int i)
break;
case OUTPUT_DATE_FMT:
status_mesg(output_datefmt_str, "");
strncpy(buf, conf.output_datefmt,
strlen(conf.output_datefmt) + 1);
strncpy(buf, conf.output_datefmt, BUFSIZ);
buf[BUFSIZ - 1] = '\0';
if (updatestring(win[STA].p, &buf, 0, 1) == 0) {
strncpy(conf.output_datefmt, buf,
strlen(buf) + 1);
strncpy(conf.output_datefmt, buf, BUFSIZ);
conf.output_datefmt[BUFSIZ - 1] = '\0';
}
break;
case INPUT_DATE_FMT:
@ -755,9 +755,11 @@ static void general_option_edit(int i)
break;
case DAY_HEADING_FMT:
status_mesg(output_datefmt_str, "");
strcpy(buf, conf.day_heading);
strncpy(buf, conf.day_heading, BUFSIZ);
buf[BUFSIZ - 1] = '\0';
if (updatestring(win[STA].p, &buf, 0, 1) == 0) {
strcpy(conf.day_heading, buf);
strncpy(conf.day_heading, buf, BUFSIZ);
conf.output_datefmt[BUFSIZ - 1] = '\0';
}
break;
}