bug#21250: sed: empty label for :/b/t commands

[Jim Meyering]

On Sun, Aug 23, 2015 at 5:08 AM, Stephane Chazelas
<address@hidden> wrote:
> 2015-08-22 22:02:42 -0700, Jim Meyering:
> [...]
>> I am preparing a patch to remove that functionality, making sed do
>> this, instead:
>>
>>   $ sed :
>>   sed: -e expression #1, char 1: ":" lacks a label
>>
>> Does anyone feel differently?
>> However, note that I have not surveyed other sed implementations.
> [...]
>
> Thanks.
>
> Solaris 11 (/bin/sed or /usr/xpg4/bin/sed):
>
> $ echo aa | sed -e : -e s/a/b/ -e t
> ba
>
> (it doesn't complain on :, but t branches to the end, not to
> that empty label).
>
> Same with busybox sed.
>
> With FreeBSD sed:
>
> $ echo aa | sed -e : -e s/a/b/ -e t
> sed: 1: ":
> ": empty label
>
> (exit status: 1)

Thanks for checking those.
Here's the patch I expect to push tomorrow:

0001-sed-reject-a-command-without-a-label.patch
Description: Text Data