Skip to content

sudoers: accept tab as command/argument separator - #1670

Merged
squell merged 1 commit into
trifectatechfoundation:mainfrom
oiqgj:fix/sudoers-command-token-tab
Aug 24, 2026
Merged

sudoers: accept tab as command/argument separator#1670
squell merged 1 commit into
trifectatechfoundation:mainfrom
oiqgj:fix/sudoers-command-token-tab

Conversation

@oiqgj

@oiqgj oiqgj commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Fixes #1669

Fixes an issue where tabs were not treated as valid separators between command tokens when parsing sudoers command specifications.

This changes the parser to treat \t the same way as a space when accepting command tokens.

VedantMadane added a commit to VedantMadane/sudo-rs that referenced this pull request Aug 24, 2026
Previous run: 824 passed; flaky sudo::syslog::sudo_logs_every_executed_command
(empty journal capture). TAB fix is unit-tested; sibling PR trifectatechfoundation#1670 passed the
same compliance suite with an equivalent accept() change.

Signed-off-by: Vedant Madane <6527493+VedantMadane@users.noreply.github.com>
@squell

squell commented Aug 24, 2026

Copy link
Copy Markdown
Member

Thanks!

@squell
squell merged commit 4df2904 into trifectatechfoundation:main Aug 24, 2026
20 checks passed
VedantMadane added a commit to VedantMadane/sudo-rs that referenced this pull request Aug 24, 2026
Previous run: 824 passed; flaky sudo::syslog::sudo_logs_every_executed_command
(empty journal capture). TAB fix is unit-tested; sibling PR trifectatechfoundation#1670 passed the
same compliance suite with an equivalent accept() change.

Signed-off-by: Vedant Madane <6527493+VedantMadane@users.noreply.github.com>
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.

Command does not accept TAB characters

2 participants