diff options
| author | Toomas Soome <tsoome@me.com> | 2021-03-18 00:27:26 +0200 |
|---|---|---|
| committer | Toomas Soome <tsoome@me.com> | 2021-03-26 17:59:09 +0200 |
| commit | d4d62540c939a1f768b551ce50297d66799bd53e (patch) | |
| tree | 877e084f3092a8409861de7750a09d45ee854397 /usr/src | |
| parent | 49950d9c675502476d8644b1e5bb1bf0d743f53b (diff) | |
| download | illumos-joyent-d4d62540c939a1f768b551ce50297d66799bd53e.tar.gz | |
13652 xargs: writing 1 byte into a region of size 0
Reviewed by: C Fraire <cfraire@me.com>
Reviewed by: Andy Fiddaman <andy@omnios.org>
Approved by: Robert Mustacchi <rm@fingolfin.org>
Diffstat (limited to 'usr/src')
| -rw-r--r-- | usr/src/cmd/xargs/xargs.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/usr/src/cmd/xargs/xargs.c b/usr/src/cmd/xargs/xargs.c index ba582df141..d16ff2fc16 100644 --- a/usr/src/cmd/xargs/xargs.c +++ b/usr/src/cmd/xargs/xargs.c @@ -736,12 +736,12 @@ ermsg(char *messages, ...) } static int -echoargs() +echoargs(void) { char **anarg; char **tanarg; /* tmp ptr */ - int i; - char reply[LINE_MAX]; + int i; + char reply[LINE_MAX]; tanarg = anarg = arglist-1; @@ -773,7 +773,10 @@ echoargs() break; } } - reply[i] = 0; + if (i < LINE_MAX) + reply[i] = '\0'; + else + reply[LINE_MAX - 1] = '\0'; /* flush remainder of line if necessary */ if (i == LINE_MAX) { |