Bug in busybox sed - this one is real

Rich Felker dalias at aerifal.cx
Thu Jun 7 15:28:15 UTC 2012


On Thu, Jun 07, 2012 at 04:37:17PM +0200, Denys Vlasenko wrote:
> > The fix broke it worse:
> >
> > $ sed 's/\(["`$\\]\)/\\\1/g' << "EOF"
> > {\"\1\"
> > EOF
> > {\\\"\\1\\\"
> > $ busybox sed 's/\(["`$\\]\)/\\\1/g' << "EOF"
> > {\"\1\"
> > EOF
> > {\\1\\
> >
> > Previous version of busybox sed handled this case fine.
> 
> Indeed, null match code falsely triggers here.
> Fixed in git, please try...

Thanks, seems much better now!
I think this finally makes busybox sed a viable replacement for GNU
sed for general-purpose use (including for a system that can rebuild
itself).

Rich


More information about the busybox mailing list