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

Colon in user results in invalid qrexec command #9154

Open
DemiMarie opened this issue Apr 25, 2024 · 0 comments
Open

Colon in user results in invalid qrexec command #9154

DemiMarie opened this issue Apr 25, 2024 · 0 comments
Labels
affects-4.1 This issue affects Qubes OS 4.1. affects-4.2 This issue affects Qubes OS 4.2. C: core diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@DemiMarie
Copy link

DemiMarie commented Apr 25, 2024

How to file a helpful issue

Qubes OS release

R4.2, but this problem is probably older than R4.1.

Brief summary

If one passes --user=a:b (or any other string containing a colon) to qvm-run in dom0, the colon is interpreted as the end of the username, rather than as part of it.

Steps to reproduce

Either:

  1. Pass a username to qvm-run --user= that contains a colon.
  2. Include a user= keyword in a qrexec policy that contains a colon.

Expected behavior

Error stating that usernames in qrexec cannot contain a colon.

Actual behavior

Colon is interpreted as end of the username, and everything after it is interpreted as the command. For instance, qvm-run --user=DEFAULT:nogui is equivalent to qvm-run --no-gui.

@DemiMarie DemiMarie added T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. C: core P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. diagnosed Technical diagnosis has been performed (see issue comments). labels Apr 25, 2024
@andrewdavidwong andrewdavidwong added affects-4.1 This issue affects Qubes OS 4.1. affects-4.2 This issue affects Qubes OS 4.2. labels Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.1 This issue affects Qubes OS 4.1. affects-4.2 This issue affects Qubes OS 4.2. C: core diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

No branches or pull requests

2 participants