isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Warn with "unknown user" not "unknown uid" if user cannot be resolved.

Browse Source 
Prior to sudo 1.8 this was after a getpwuid() but now we use
getpwnam().
This commit is contained in:
Todd C. Miller
2023-03-29 14:49:41 -06:00
parent ad890acf6c
commit ac12f82d86
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
plugins/sudoers/sudoers.c
View File

@@ -917,7 +917,7 @@ init_vars(char * const envp[])
* YP/NIS/NIS+/LDAP/etc daemon has died. * YP/NIS/NIS+/LDAP/etc daemon has died.
*/ */
if (sudo_mode == MODE_KILL || sudo_mode == MODE_INVALIDATE) { if (sudo_mode == MODE_KILL || sudo_mode == MODE_INVALIDATE) {
sudo_warnx(U_("unknown uid %u"), (unsigned int) user_uid); sudo_warnx(U_("unknown user %s"), user_name);
debug_return_bool(false); debug_return_bool(false);
} }
@@ -938,8 +938,7 @@ init_vars(char * const envp[])
/* It is now safe to use log_warningx() and set_perms() */ /* It is now safe to use log_warningx() and set_perms() */
if (unknown_user) { if (unknown_user) {
log_warningx(SLOG_SEND_MAIL, N_("unknown uid %u"), log_warningx(SLOG_SEND_MAIL, N_("unknown user %s"), user_name);
(unsigned int) user_uid);
debug_return_bool(false); debug_return_bool(false);
} }