0001311: sed incorrectly parses command file - BusyBox Bug and Patch Tracking

BusyBox Bug and Patch Tracking

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0001311

[BusyBox] Other

major

always

04-12-07 14:24

04-13-07 13:11

Reporter

kiltedknight

View Status

public

Assigned To

BusyBox

Priority

normal

Resolution

fixed

Status

closed

Product Version

1.4.x

Summary

0001311: sed incorrectly parses command file

Description

When running the command "sed -f cmds.sed < input > output" with cmds.sed containing:

s/^[^:]*: *//
s/\s*\\$//
s/\s*$[-./[:alnum:]_]\+$/\1:\
/g

It will produce the error:
sed: unmatched '/'

When going through the code in sed.c, I discovered that it was attempting to parse only the "/g" portion of the final command.

Additional Information

Attached diff will fix the issue.

Attached Files

bb_sed_fix.diff [^] (394 bytes) 04-12-07 14:24

Relationships

Notes
(0002309) kiltedknight 04-12-07 15:01	This bug is against the current version (1.5.0)

(0002313) vda 04-13-07 13:11	Fixed in svd and added to 1.5.0 fixes directory. Thanks!

(0002314) vda 04-13-07 13:11	Fixed in svn and added to 1.5.0 fixes directory. Thanks!

Issue History
Date Modified	Username	Field	Change
04-12-07 14:24	kiltedknight	New Issue
04-12-07 14:24	kiltedknight	Status	new => assigned
04-12-07 14:24	kiltedknight	Assigned To	=> BusyBox
04-12-07 14:24	kiltedknight	File Added: bb_sed_fix.diff
04-12-07 14:24	kiltedknight	Issue Monitored: kiltedknight
04-12-07 15:01	kiltedknight	Note Added: 0002309
04-13-07 13:11	vda	Status	assigned => closed
04-13-07 13:11	vda	Note Added: 0002313
04-13-07 13:11	vda	Resolution	open => fixed
04-13-07 13:11	vda	Note Added: 0002314