Bug in busybox sed - this one is real

[John Spencer]

On 06/06/2012 02:24 AM, Rich Felker wrote:
> On Mon, Jun 04, 2012 at 02:56:15PM +0200, Denys Vlasenko wrote:
>> On Sun, Jun 3, 2012 at 3:54 PM, Rich Felker<dalias at aerifal.cx>  wrote:
>>> Ping. Anybody looked at this? I filed it as bug # 5246.
>> fixed in git:
>>
>> commit 21f6fbf545e7fa58f0eaa444001a9d25bc37c4eb
>> Author: Denys Vlasenko<vda.linux at googlemail.com>
>> Date:   Mon Jun 4 14:44:47 2012 +0200
>>
>>      sed: fix zero chars match/replace
> The fix broke it worse:
>
> $ sed 's/\(["`$\\]\)/\\\1/g'<<  "EOF"
> {\"\1\"
> EOF
> {\\\"\\1\\\"
> $ busybox sed 's/\(["`$\\]\)/\\\1/g'<<  "EOF"
> {\"\1\"
> EOF
> {\\1\\
i installed bb 1.20.1 and applied this single commit, then tried to 
build stuff
here's a log of the errors i encountered (different output with gnu sed 
and bb sed)

http://sprunge.us/OMgf

there's at least one more broken case than the above quoting error.

> Previous version of busybox sed handled this case fine.
>
> Rich
> _______________________________________________
> busybox mailing list
> busybox at busybox.net
> http://lists.busybox.net/mailman/listinfo/busybox
>