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

[Stefan Farfeleder]

On Thu, Oct 01, 2009 at 09:12:19PM +0200, Andre Albsmeier wrote:
> 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,

Done! (to stable/7, that is)