Skip to content

tests: yes: add test for 2nd splice(2)'s error#288

Open
oech3 wants to merge 1 commit into
coreutils:masterfrom
oech3:patch-9
Open

tests: yes: add test for 2nd splice(2)'s error#288
oech3 wants to merge 1 commit into
coreutils:masterfrom
oech3:patch-9

Conversation

@oech3

@oech3 oech3 commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

tests: yes: add test for 2nd splice(2)'s error

Comment thread tests/misc/yes.sh
if no_zero_copy true; then
test "$(no_zero_copy yes | head -n2 | paste -s -d '')" = 'yy' || fail=1
# however, 2nd+ error should be catched
returns_ 1 strace -o /dev/null -e inject=splice:error=EINTR:when=3 $SHELL -c "timeout 1 yes>/dev/null" || fail=1

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not add other syscalls since fallback from tee(2) is possible if middler pipe is always used.

* tests/musc/yes.sh Check that 2nd splice(2)'s EINTR is catched
coreutils#268
@oech3 oech3 changed the title tests, yes: add test for 2nd splice(2)'s error tests: yes: add test for 2nd splice(2)'s error Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant