GNU bug report logs - #18266
grep -P and invalid exits with error

Previous Next

Package: grep;

Reported by: Santiago <santiago <at> debian.org>

Date: Thu, 14 Aug 2014 15:43:02 UTC

Severity: wishlist

Merged with 18455

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

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: Vincent Lefevre <vincent <at> vinc17.net>
To: Paul Eggert <eggert <at> cs.ucla.edu>, 758105 <at> bugs.debian.org
Cc: Santiago <santiago <at> debian.org>, 18266 <at> debbugs.gnu.org
Subject: bug#18266: Bug#758105: bug#18266: grep -P and invalid exits with error
Date: Thu, 14 Aug 2014 19:44:42 +0200

On 2014-08-14 09:15:58 -0700, Paul Eggert wrote:
> That commit was necessary to avoid undefined behavior in libpcre.  We can't
> simply undo the commit (unless you want to reintroduce security holes into
> grep :-).  The current behavior is the best we can do, unless someone fixes
> libpcre (which doesn't appear to be likely), or unless someone takes the
> time to write code in grep to work around the problem.
> 
> One way forward is suggested in <http://bugs.gnu.org/17245#43>.  No doubt
> there are others.  Can you suggest a volunteer to take this on?

Discarding input lines with invalid UTF-8 sequences is not OK.
IMHO, it would be better to replace invalid UTF-8 sequences by
zero bytes before passing them to libpcre. Is it allowed to do
that in Pexecute()?

-- 
Vincent Lefèvre <vincent <at> vinc17.net> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)
This bug report was last modified 10 years and 248 days ago.

Previous Next

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