about summary refs log tree commit diff
path: root/ui-ssdiff.c
diff options
context:
space:
mode:
authorChristian Hesse <mail@eworm.de>2018-08-28 18:23:36 +0200
committerChristian Hesse <mail@eworm.de>2018-09-11 08:47:12 +0200
commita96f2890f41e0b9b0ffa1bcdb1dddbef28c01662 (patch)
treed88340d58313c6f20e063003d809e158c7295c29 /ui-ssdiff.c
parentui-ssdiff: ban strncpy() (diff)
downloadcgit-magenta-a96f2890f41e0b9b0ffa1bcdb1dddbef28c01662.tar.xz
ui-ssdiff: ban strcat()
Git upstream bans strcat() with commit:

  banned.h: mark strcat() as banned
  1b11b64b815db62f93a04242e4aed5687a448748

Signed-off-by: Christian Hesse <mail@eworm.de>
Diffstat (limited to 'ui-ssdiff.c')
-rw-r--r--ui-ssdiff.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/ui-ssdiff.c b/ui-ssdiff.c

index a3dd059..c456033 100644 --- a/ui-ssdiff.c +++ b/ui-ssdiff.c
@@ -117,6 +117,7 @@ static char *replace_tabs(char *line) int n_tabs = 0; int i; char *result; + int result_len; if (linelen == 0) { result = xmalloc(1); @@ -128,13 +129,14 @@ static char *replace_tabs(char *line) if (line[i] == '\t') n_tabs += 1; } - result = xmalloc(linelen + n_tabs * 8 + 1); + result_len = linelen + n_tabs * 8; + result = xmalloc(result_len + 1); result[0] = '\0'; for (;;) { cur_buf = strchr(prev_buf, '\t'); if (!cur_buf) { - strcat(result, prev_buf); + strncat(result, prev_buf, result_len); break; } else { strncat(result, prev_buf, cur_buf - prev_buf);