Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

builtins ignore signals with redirections #7228

Open
zanchey opened this issue Jul 27, 2020 · 1 comment
Open

builtins ignore signals with redirections #7228

zanchey opened this issue Jul 27, 2020 · 1 comment
Milestone

Comments

@zanchey
Copy link
Member

zanchey commented Jul 27, 2020

As noted by @krobelus and me on Gitter, some builtins cannot be cancelled. This effectively livelocks the process running them, potentially infinitely.

Examples include:

  • count </dev/urandom
  • string trim </dev/stdin and math </dev/stdin, "because they use read_blocked which ignores EINTR from read(2). They ignore Control-C since fd6d814, which seems weird."
@zanchey zanchey added this to the fish-future milestone Jul 27, 2020
@ridiculousfish
Copy link
Member

Yeah nice find!

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

No branches or pull requests

2 participants