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

running command in any VM and interrupting doesn't change the running command in the VM. (so e.g. firefox will continue running) #4580

Open
marmarek opened this issue Dec 6, 2018 · 0 comments
Labels
C: core P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality.

Comments

@marmarek
Copy link
Member

marmarek commented Dec 6, 2018

Right now qrexec protocol does not support "interrupt" signal of any kind, which is intentional in most cases. qrexec is meant as a transport layer between domains potentially with different trust levels. Interrupting a service in wrong time may have serious implications, including security issues.
qubes.VMShell service, especially when called from dom0, is somehow special case here, because

  1. most shell command are expecting to be interrupted with SIGINT/Ctrl+C if necessary
  2. this service already gives the ultimate control over its target, so nothing worse can happen

Maybe the way forward would be a per-service configuration if given service call can be interrupted - by default enabled only for qubes.VMShell. But this is not a simple change, involve protocol change etc.

Originally posted by @marmarek in https://github.com/QubesOS/qubes-issues/issue_comments#issuecomment-445056975

@marmarek marmarek added T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality. C: core labels Dec 6, 2018
@marmarek marmarek added this to the Release 4.1 milestone Dec 6, 2018
@andrewdavidwong andrewdavidwong removed this from the Release 4.2 milestone Aug 13, 2023
@andrewdavidwong andrewdavidwong added the P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. label Aug 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: core P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality.
Projects
None yet
Development

No branches or pull requests

2 participants