diff options
author | Dominique Martinet | 2023-09-19 17:11:02 +0900 |
---|---|---|
committer | Denys Vlasenko | 2024-01-02 00:58:56 +0100 |
commit | 5dc9ece3b9e87af0dcb01449821ac827391ac116 (patch) | |
tree | d793f9ebff0316209d063aea9ff3e7d352949520 /shell/ash_test | |
parent | 1cac2585217f830d29f7e9d2a1226d55c80886b6 (diff) | |
download | busybox-5dc9ece3b9e87af0dcb01449821ac827391ac116.zip busybox-5dc9ece3b9e87af0dcb01449821ac827391ac116.tar.gz |
sed: check errors writing file with sed -i
sed would currently not error if write failed when modifying a file.
This can be reproduced with the following 'script':
$ sudo mount -t tmpfs tmpfs -o size=1M /tmp/m
$ sudo chmod 777 /tmp/m
$ echo foo > /tmp/m/foo
$ dd if=/dev/zero of=/tmp/m/fill bs=4k
dd: error writing '/tmp/m/fill': No space left on device
256+0 records in
255+0 records out
1044480 bytes (1.0 MB, 1020 KiB) copied, 0.00234567 s, 445 MB/s
$ busybox sed -i -e 's/.*/bar/' /tmp/m/foo
$ echo $?
0
$ cat /tmp/m/foo
<empty>
new behaviour:
$ echo foo > /tmp/m/foo
$ ./busybox sed -i -e 's/.*/bar/' /tmp/m/foo
sed: write error
$ echo $?
4
$ cat /tmp/m/foo
foo
function old new delta
sed_main 754 801 +47
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/0 up/down: 47/0) Total: 47 bytes
text data bss dec hex filename
75727 2510 1552 79789 137ad busybox_old
75774 2510 1552 79836 137dc busybox_unstripped
Signed-off-by: Dominique Martinet <dominique.martinet@atmark-techno.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/ash_test')
0 files changed, 0 insertions, 0 deletions