[Clfs-dev] Grep 2.20

William Harrington kb0iic at berzerkula.org
Tue Jun 3 15:32:29 PDT 2014


Greetings,

We are in a package freeze with Grep 2.19.

For 2.20, are any of the following a required upgrade for grep which  
would be a critical issue for our release?

2014-06-03  Jim Meyering  <meyering at fb.com>

         version 2.20
         * NEWS: Record release date.

2014-05-30  Jim Meyering  <meyering at fb.com>

         grep: fix --max-count=N (-m N) to stop reading after Nth match
         With --max-count=N (-m N), grep is supposed to stop reading  
input
         after it has found the Nth match.  However, a recent context-
         related change made it so grep would always read to end of  
file.
         * src/grep.c (prtext): Don't let a negative "out_after" value
         make "pending" line count negative.
         * tests/max-count-overread: New test, for this.
         * tests/Makefile.am (TESTS): Add it.
         * NEWS (Bug fixes): Mention it.
         * THANKS: Add names of two recent bug reporters.
         This bug was introduced by commit v2.18-139-g5122195.
         Reported by Marc Aldorasi in http://bugs.gnu.org/17640.

2014-05-29  Jim Meyering  <meyering at fb.com>

         dfa: fix off-by-one under-allocation from recent change
         Commit v2.19-10-gc32ff67 mistakenly made this change:
           -realloc_trans_if_necessary (d, 1);
           +realloc_trans_if_necessary (d, 0);
         which led to a heap buffer overflow.
         * src/dfa.c (dfaexec): Allocate space for one state, as before.

2014-05-28  Norihiro Tanaka  <noritnk at kcn.ne.jp>

         dfa: fix bug with regex containing multiple begin/end-line  
constraints
         grep -E 'a(b$|c$)' would mistakenly match "aa".
         * src/dfa.c (dfamust): When resetting 'is' in OR, also reset
         'begline' and 'endline' of 'must'.
         * NEWS (Bug fixes): Mention it.
         This bug was introduced via commit v2.18-85-g2c94326.
         Reported by Péter Radics in <http://bugs.gnu.org/17617>.

2014-05-26  Norihiro Tanaka  <noritnk at kcn.ne.jp>

         dfa: simplify building initial state
         build_state_zero doesn't need the struct dfa to be initialized,
         so remove the initialization and simplify.
         * src/dfa.c (build_state_zero): Remove.
         (dfaexec): Call realloc_trans_if_necessary and build_state  
directly.

         dfa: revert "grep: do not count newline before the start of  
buffer"
         This reverts commit 5dc3af2806d21455b818be3f9da26c372e4a7f8d.
         The previous change renders that commit unnecessary.

Sincerely,

William Harrington


More information about the Clfs-dev mailing list