GNU bug report logs - #19563
grep -F: fix a heap buffer (read) overrun

Previous Next

Package: grep;

Reported by: Jim Meyering <jim <at> meyering.net>

Date: Sat, 10 Jan 2015 23:44:02 UTC

Severity: normal

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: bug-grep <at> gnu.org
Cc: Yuliy Pisetsky <ypisetsky <at> fb.com>, Nima Aghdaii <naghdaii <at> fb.com>
Subject: grep -F: fix a heap buffer (read) overrun
Date: Sat, 10 Jan 2015 15:43:02 -0800

[Message part 1 (text/plain, inline)]
Colleagues Nima Aghdaii and Yuliy Pisetsky found and fixed a heap
buffer overrun in grep's kwset.c.  The underlying bug, already hard to
trigger, would most often result in a "mere" heap UMR (uninitialized
memory read), but Yuliy constructed inputs (see the new test) that
cause a buffer overrun.

I'm attaching two other related patches:
  - grep: avoid false-positive UMR
  - tests: add support for ASAN memory poisoning

I expect to push these by Monday.

[0001-grep-avoid-false-positive-UMR.patch (application/octet-stream, attachment)]
[0002-grep-F-fix-a-heap-buffer-read-overrun.patch (application/octet-stream, attachment)]
[0003-tests-add-support-for-ASAN-memory-poisoning.patch (application/octet-stream, attachment)]
This bug report was last modified 10 years and 53 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.