| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
- Reset the hold-space in between files
- quit the editor as soon as a 'q' command is found
- Make sure the temp-file is written back to the original file if we quit
the editor
temp-file not written back issue found by Time Chase.
Lots of feedback from millert@ and schwarze@
OK millert@
|
| |
|
|
|
|
|
|
| |
From Daniel Cegielka - thanks
ok jca@
|
|
|
|
| |
rectification.
|
|
|
|
|
|
| |
appease gcc.
ok procter@ deraadt@
|
|
|
|
|
|
| |
By pfg@ from FreeBSD
OK millert@ and otto@
|
|
|
|
|
|
|
|
|
|
| |
of word mismatch as reported by jsg@.
Discussed with otto@ and others early on, earlier version tested in ports build
by aja@
OK millert@
Few readability tweaks and OK schwarze@
|
|
|
|
| |
Discussed with jca@.
|
| |
|
|
|
|
| |
ok tedu miod
|
|
|
|
| |
feedback/ok deraadt@ millert@
|
|
|
|
|
| |
handling along the way.
Reviews by Brendan MacDonell, Jeremy Devenport, florian, doug, millert
|
|
|
|
|
| |
to size_t to avoid overflow after allocation success
ok guenther doug
|
|
|
|
|
| |
clearer and avoid a crash on 'g;P' found by Sebastien Marie with
the afl fuzzer.
|
| |
|
|
|
|
| |
ok krw okan
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
issues regarding the replacement of zero-length strings.
This commit brings back rev. 1.16, but without the regression that
forced the backout: No NUL bytes will be output now, not even when
the input file lacks a trailing newline character and there is a
zero-length match at the end.
OK otto@ deraadt@;
and naddy@ (who originally found the regression) checked that
the regression is indeed fixed.
|
|
|
|
|
|
|
|
| |
When the input does not end in a trailing newline character
and there is an empty match at the end, the new code adds
a spurious '\0' character.
I have a fix, but otto@ prefers backout and full re-evaluation
after release.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
lines and simplifying it by removing the switch statement implementing
/g, /1, and /2 separately and repetitively.
The idea to make the loop control variable slen, i.e. the length of the
string remaining to be processed, signed, and stay in the loop even when
slen == 0 (i.e. at the end of the string), lifted from FreeBSD by otto@.
On i386, process.o shrinks by 440 bytes, and the sed binary by 23 bytes.
This fixes multiple aspects of the replacement of multiple (/g) or
specific (e.g. /2) instances of zero-length matches, both with BREs and
EREs, both with and without a trailing newline character on the input.
Feedback and OK otto@.
|
|
|
|
|
|
|
| |
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms
|
|
|
|
| |
backspace is escaped as \b. OK phessler@ and jmc@
|
|
|
|
| |
an if without braces, or worse, into a return.
|
|
|
|
| |
ok millert@ otto@
|
| |
|
|
|
|
| |
rescinded 22 July 1999. Proofed by myself and Theo.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
is not specified, and then the first part of the pattern space is
deleted, when there are two or more input lines, as this results
in subtraction of one from an unsigned integral value of '0'. That
bogus value is used in one case for a loop (that will run far too
many times in this case) and a function to search for a value within
a specified range of memory, however now the range of memory is
obscenely large and a segmentation (or memory) fault will occur.
This is fixed by checking for and appropriately handling a nil
pattern space as if the specified search in memory failed, as indeed
it obviously will with nil pattern space.
From Tim J. Robbins by way of FreeBSD
|
| |
|
|
|
|
| |
millert@ ok
|
| |
|
| |
|
| |
|
| |
|
|
|