Bug in 7.2-STABLE's /bin/sh?

Andre Albsmeier Andre.Albsmeier at siemens.com
Thu Oct 1 19:31:49 UTC 2009


On Thu, 01-Oct-2009 at 20:31:01 +0200, Andre Albsmeier wrote:
> On Thu, 01-Oct-2009 at 18:44:08 +0300, Andriy Gapon wrote:
> > on 01/10/2009 17:49 Andre Albsmeier said the following:
> > > Hello all,
> > > 
> > > is it correct to print OK here?
> > > 
> > > ------------------ snip ------------------
> > > 
> > > #!/bin/sh
> > > 
> > > if false || ! echo bla | grep -q bla; then
> > >   echo OK
> > > fi
> > > 
> > > ------------------ snap -------------------
> > > 
> > > 7.2-STABLE (can't check others at the moment)
> > > does which I think is wrong...
> > 
> > This looks like a bug and it seems to be fixed in head.
> > Forgotten MFC?
> 
> Have you got a PR# handy?

Found it myself:

http://www.freebsd.org/cgi/cvsweb.cgi/src/bin/sh/parser.c.diff?r1=1.60;r2=1.61

seems to be it. Could someone please MFC that to 7.2?

Thanks,

	-Andre


More information about the freebsd-stable mailing list