Skip to content

Make Digits tokenizer more precise#1603

Merged
squell merged 2 commits into
mainfrom
fix-large-gid_uid
May 26, 2026
Merged

Make Digits tokenizer more precise#1603
squell merged 2 commits into
mainfrom
fix-large-gid_uid

Conversation

@squell
Copy link
Copy Markdown
Member

@squell squell commented May 26, 2026

We use to cap this at 10e9 so the str::parse method would always succeed; now we cap it at 10 digits and simply listen to the result of str::parse.

Closes #1602

@squell squell added the C-parser Parser/AST label May 26, 2026
@squell squell marked this pull request as draft May 26, 2026 10:23
Comment thread src/sudoers/tokens.rs Outdated
@squell squell force-pushed the fix-large-gid_uid branch from 635cf4e to b22f49b Compare May 26, 2026 10:35
@squell squell marked this pull request as ready for review May 26, 2026 10:35
@squell squell enabled auto-merge May 26, 2026 10:37
@squell squell merged commit 09a5b9a into main May 26, 2026
20 checks passed
@squell squell deleted the fix-large-gid_uid branch May 26, 2026 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-parser Parser/AST

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Large group id's / user id's (larger than 10 digits but < 32bits) are not parsed correctly.

2 participants