d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

bsdgrep: correct assumptions to prepare for chunking

Browse Source 
Correct a couple of minor BSD grep assumptions that are valid for line
processing but not future chunk-based processing.

Submitted by:	Kyle Evans <kevans91@ksu.edu>
Reviewed by:	bapt, cem
Differential Revision:	https://reviews.freebsd.org/D10824
This commit is contained in:
Ed Maste 2017-05-26 02:30:26 +00:00
parent 0523111725
commit 8bf4606408
3 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
usr.bin/grep/grep.c
View File

@ -78,7 +78,7 @@ const char *errstr[] = {
};
/* Flags passed to regcomp() and regexec() */
int cflags = REG_NOSUB;
int cflags = REG_NOSUB | REG_NEWLINE;
int eflags = REG_STARTEND;
/* XXX TODO: Get rid of this flag.

2
usr.bin/grep/grep.h
View File

@ -82,7 +82,7 @@ extern const char *errstr[];
#define EXCL_PAT 0
#define INCL_PAT 1
#define MAX_LINE_MATCHES 32
#define MAX_MATCHES 32
struct file {
int fd;

8
usr.bin/grep/util.c
View File

@ -61,7 +61,7 @@ static bool first_match = true;
* other useful bits
*/
struct parsec {
regmatch_t matches[MAX_LINE_MATCHES]; /* Matches made */
regmatch_t matches[MAX_MATCHES]; /* Matches made */
struct str ln; /* Current line */
size_t lnstart; /* Position in line */
size_t matchidx; /* Latest match index */
@ -295,7 +295,7 @@ procfile(const char *fn)
/* Print the matching line, but only if not quiet/binary */
if (t == 0 && printmatch) {
printline(&pc, ':');
while (pc.matchidx >= MAX_LINE_MATCHES) {
while (pc.matchidx >= MAX_MATCHES) {
/* Reset matchidx and try again */
pc.matchidx = 0;
if (procline(&pc) == 0)
@ -401,7 +401,7 @@ procline(struct parsec *pc)
lastmatches = 0;
startm = matchidx;
retry = 0;
if (st > 0)
if (st > 0 && pc->ln.dat[st - 1] != fileeol)
leflags |= REG_NOTBOL;
/* Loop to compare with all the patterns */
for (i = 0; i < patterns; i++) {
@ -483,7 +483,7 @@ procline(struct parsec *pc)
}
/* avoid excessive matching - skip further patterns */
if ((color == NULL && !oflag) || qflag || lflag ||
matchidx >= MAX_LINE_MATCHES) {
matchidx >= MAX_MATCHES) {
pc->lnstart = nst;
lastmatches = 0;
break;