Rewrite -c and -b processing to shrink code and eliminate buffer overrun.

This commit is contained in:
Matt Kraai 2001-05-05 16:19:13 +00:00
parent 6b8c550d88
commit 2e6c878763
2 changed files with 8 additions and 28 deletions

View File

@ -108,20 +108,10 @@ static void cut_file(FILE *file)
for (line = NULL; (line = get_line_from_file(file)) != NULL; free(line)) { for (line = NULL; (line = get_line_from_file(file)) != NULL; free(line)) {
/* cut based on chars/bytes */ /* cut based on chars/bytes */
if (part == 'c' || part == 'b') { if (part == 'c' || part == 'b') {
int i; chomp(line);
/* a valid end position has been specified */ if (0 < endpos && endpos < strlen(line))
if (endpos > 0) { line[endpos] = '\0';
for (i = startpos-1; i < endpos; i++) { puts(line + startpos - 1);
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);
}
}
} }
/* cut based on fields */ /* cut based on fields */
else if (part == 'f') { else if (part == 'f') {

18
cut.c
View File

@ -108,20 +108,10 @@ static void cut_file(FILE *file)
for (line = NULL; (line = get_line_from_file(file)) != NULL; free(line)) { for (line = NULL; (line = get_line_from_file(file)) != NULL; free(line)) {
/* cut based on chars/bytes */ /* cut based on chars/bytes */
if (part == 'c' || part == 'b') { if (part == 'c' || part == 'b') {
int i; chomp(line);
/* a valid end position has been specified */ if (0 < endpos && endpos < strlen(line))
if (endpos > 0) { line[endpos] = '\0';
for (i = startpos-1; i < endpos; i++) { puts(line + startpos - 1);
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);
}
}
} }
/* cut based on fields */ /* cut based on fields */
else if (part == 'f') { else if (part == 'f') {