emo/shadow
1
0
Fork 0
Code Issues Pull Requests Packages Releases Activity

pwck.c: only check home dirs if set and not a system user

Browse Source 
Closes #126

Changelog: pwck, better to look at array than to use strnlen.
This commit is contained in:
ed 2019-08-25 20:11:24 +01:00 committed by Serge Hallyn
parent 5687be5f31
commit c4e8b411d4
1 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/pwck.c
View File

@ -382,6 +382,8 @@ static void check_pw_file (int *errors, bool *changed)
struct commonio_entry *pfe, *tpfe; struct commonio_entry *pfe, *tpfe;
struct passwd *pwd; struct passwd *pwd;
struct spwd *spw; struct spwd *spw;
uid_t min_sys_id = (uid_t) getdef_ulong ("SYS_UID_MIN", 101UL);
uid_t max_sys_id = (uid_t) getdef_ulong ("SYS_UID_MAX", 999UL);
/* /*
* Loop through the entire password file. * Loop through the entire password file.
@ -510,15 +512,20 @@ static void check_pw_file (int *errors, bool *changed)
} }
/* /*
* Make sure the home directory exists * If uid is system and has a home directory, then check
*/ */
if (!quiet && (access (pwd->pw_dir, F_OK) != 0)) { if (!(pwd->pw_uid >= min_sys_id && pwd->pw_uid <= max_sys_id && pwd->pw_dir && pwd->pw_dir[0])) {
/* /*
* Home directory doesn't exist, give a warning * Make sure the home directory exists
*/ */
printf (_("user '%s': directory '%s' does not exist\n"), if (!quiet && (access (pwd->pw_dir, F_OK) != 0)) {
pwd->pw_name, pwd->pw_dir); /*
*errors += 1; * Home directory doesn't exist, give a warning
*/
printf (_("user '%s': directory '%s' does not exist\n"),
pwd->pw_name, pwd->pw_dir);
*errors += 1;
}
} }
/* /*