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

src/useradd.c: E_BAD_NAME: Use a different error code for bad login names #1141

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alejandro-colomar
Copy link
Collaborator

Wrappers like adduser(8) want to do their own stuff if the login name is
bad.  For that, they need to be able to differentiate such an error.

Closes: #1103
Suggested-by: @zeha
Cc: @Zugschlus

@alejandro-colomar alejandro-colomar changed the title src/useradd.c: E_BAD_NAME: Use a different error code for bad loginames src/useradd.c: E_BAD_NAME: Use a different error code for bad login names Dec 5, 2024
@alejandro-colomar alejandro-colomar force-pushed the badname branch 2 times, most recently from c290b35 to a015e91 Compare December 6, 2024 11:53
…ames

Wrappers like adduser(8) want to do their own stuff if the login name is
bad.  For that, they need to be able to differentiate such an error.

Closes: <shadow-maint#1103>
Suggested-by: Chris Hofstaedtler <zeha@debian.org>
Cc: Marc 'Zugschlus' Haber <mh+githubvisible@zugschlus.de>
Signed-off-by: Alejandro Colomar <alx@kernel.org>
@alejandro-colomar alejandro-colomar marked this pull request as ready for review December 22, 2024 12:00
@alejandro-colomar alejandro-colomar marked this pull request as draft December 23, 2024 09:47
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.

please consider giving a dedicated exit value for "invalid user name"
1 participant