bug in busybox sed with non-ascii chars

[Natanael Copa]

Hi,

I came across a bug (or feature) in busybox sed when trying to build firefox-29.

Testcase based on what firefox's configure scripts does:

ASCII='AA'
NONASCII=$'\246\246'

echo -e "($ASCII)\n($NONASCII)" | busybox sed 's/$/,/'


Expected result is a comma (,) after both lines. Actual result is that
the line with non-ascii does not get any comma.

With gnu sed both lines gets a trailing comma.

BusyBox v1.22.1 compiled against musl libc.

Ideas?

-nc