diff options
author | Matt Kraai <kraai@debian.org> | 2001-05-05 20:19:13 +0400 |
---|---|---|
committer | Matt Kraai <kraai@debian.org> | 2001-05-05 20:19:13 +0400 |
commit | 2e6c87876353cf5e038d8430c076f645be51da75 (patch) | |
tree | 0884fd34c92feae004fa616b323366ab97d3a24e /coreutils/cut.c | |
parent | 6b8c550d88e87c46bd8b168920492b54bdc9dc9f (diff) |
Rewrite -c and -b processing to shrink code and eliminate buffer overrun.
Diffstat (limited to 'coreutils/cut.c')
-rw-r--r-- | coreutils/cut.c | 18 |
1 files changed, 4 insertions, 14 deletions
diff --git a/coreutils/cut.c b/coreutils/cut.c index d852ab3be..efcb325df 100644 --- a/coreutils/cut.c +++ b/coreutils/cut.c @@ -108,20 +108,10 @@ static void cut_file(FILE *file) for (line = NULL; (line = get_line_from_file(file)) != NULL; free(line)) { /* cut based on chars/bytes */ if (part == 'c' || part == 'b') { - int i; - /* a valid end position has been specified */ - if (endpos > 0) { - for (i = startpos-1; i < endpos; i++) { - fputc(line[i], stdout); - } - fputc('\n', stdout); - } - /* otherwise, just go to the end of the line */ - else { - for (i = startpos-1; line[i]; i++) { - fputc(line[i], stdout); - } - } + chomp(line); + if (0 < endpos && endpos < strlen(line)) + line[endpos] = '\0'; + puts(line + startpos - 1); } /* cut based on fields */ else if (part == 'f') { |