diff --git a/src/shared/userdb.c b/src/shared/userdb.c index 0d471a2f94..e4a04123c0 100644 --- a/src/shared/userdb.c +++ b/src/shared/userdb.c @@ -27,6 +27,7 @@ typedef enum LookupWhat { struct UserDBIterator { LookupWhat what; + UserDBFlags flags; Set *links; bool nss_covered:1; bool nss_iterating:1; @@ -92,7 +93,7 @@ UserDBIterator* userdb_iterator_free(UserDBIterator *iterator) { return mfree(iterator); } -static UserDBIterator* userdb_iterator_new(LookupWhat what) { +static UserDBIterator* userdb_iterator_new(LookupWhat what, UserDBFlags flags) { UserDBIterator *i; assert(what >= 0); @@ -104,6 +105,7 @@ static UserDBIterator* userdb_iterator_new(LookupWhat what) { *i = (UserDBIterator) { .what = what, + .flags = flags, }; return i; @@ -608,7 +610,7 @@ int userdb_by_name(const char *name, UserDBFlags flags, UserRecord **ret) { if (r < 0) return r; - iterator = userdb_iterator_new(LOOKUP_USER); + iterator = userdb_iterator_new(LOOKUP_USER, flags); if (!iterator) return -ENOMEM; @@ -655,7 +657,7 @@ int userdb_by_uid(uid_t uid, UserDBFlags flags, UserRecord **ret) { if (r < 0) return r; - iterator = userdb_iterator_new(LOOKUP_USER); + iterator = userdb_iterator_new(LOOKUP_USER, flags); if (!iterator) return -ENOMEM; @@ -693,7 +695,7 @@ int userdb_all(UserDBFlags flags, UserDBIterator **ret) { assert(ret); - iterator = userdb_iterator_new(LOOKUP_USER); + iterator = userdb_iterator_new(LOOKUP_USER, flags); if (!iterator) return -ENOMEM; @@ -738,10 +740,15 @@ int userdb_iterator_get(UserDBIterator *iterator, UserRecord **ret) { if (pw->pw_uid == UID_NOBODY) iterator->synthesize_nobody = false; - r = nss_spwd_for_passwd(pw, &spwd, &buffer); - if (r < 0) { - log_debug_errno(r, "Failed to acquire shadow entry for user %s, ignoring: %m", pw->pw_name); - incomplete = ERRNO_IS_PRIVILEGE(r); + if (!FLAGS_SET(iterator->flags, USERDB_AVOID_SHADOW)) { + r = nss_spwd_for_passwd(pw, &spwd, &buffer); + if (r < 0) { + log_debug_errno(r, "Failed to acquire shadow entry for user %s, ignoring: %m", pw->pw_name); + incomplete = ERRNO_IS_PRIVILEGE(r); + } + } else { + r = -EUCLEAN; + incomplete = true; } r = nss_passwd_to_user_record(pw, r >= 0 ? &spwd : NULL, ret); @@ -750,6 +757,8 @@ int userdb_iterator_get(UserDBIterator *iterator, UserRecord **ret) { if (ret) (*ret)->incomplete = incomplete; + + iterator->n_found++; return r; } @@ -774,11 +783,11 @@ int userdb_iterator_get(UserDBIterator *iterator, UserRecord **ret) { iterator->n_found++; return synthetic_nobody_user_build(ret); } - } - /* if we found at least one entry, then ignore errors and indicate that we reached the end */ - if (r < 0 && iterator->n_found > 0) - return -ESRCH; + /* if we found at least one entry, then ignore errors and indicate that we reached the end */ + if (iterator->n_found > 0) + return -ESRCH; + } return r; } @@ -812,7 +821,7 @@ int groupdb_by_name(const char *name, UserDBFlags flags, GroupRecord **ret) { if (r < 0) return r; - iterator = userdb_iterator_new(LOOKUP_GROUP); + iterator = userdb_iterator_new(LOOKUP_GROUP, flags); if (!iterator) return -ENOMEM; @@ -856,7 +865,7 @@ int groupdb_by_gid(gid_t gid, UserDBFlags flags, GroupRecord **ret) { if (r < 0) return r; - iterator = userdb_iterator_new(LOOKUP_GROUP); + iterator = userdb_iterator_new(LOOKUP_GROUP, flags); if (!iterator) return -ENOMEM; @@ -893,7 +902,7 @@ int groupdb_all(UserDBFlags flags, UserDBIterator **ret) { assert(ret); - iterator = userdb_iterator_new(LOOKUP_GROUP); + iterator = userdb_iterator_new(LOOKUP_GROUP, flags); if (!iterator) return -ENOMEM; @@ -908,8 +917,8 @@ int groupdb_all(UserDBFlags flags, UserDBIterator **ret) { setgrent(); iterator->nss_iterating = true; - } if (r < 0) - return r; + } else if (r < 0) + return r; *ret = TAKE_PTR(iterator); return 0; @@ -936,10 +945,15 @@ int groupdb_iterator_get(UserDBIterator *iterator, GroupRecord **ret) { if (gr->gr_gid == GID_NOBODY) iterator->synthesize_nobody = false; - r = nss_sgrp_for_group(gr, &sgrp, &buffer); - if (r < 0) { - log_debug_errno(r, "Failed to acquire shadow entry for group %s, ignoring: %m", gr->gr_name); - incomplete = ERRNO_IS_PRIVILEGE(r); + if (!FLAGS_SET(iterator->flags, USERDB_AVOID_SHADOW)) { + r = nss_sgrp_for_group(gr, &sgrp, &buffer); + if (r < 0) { + log_debug_errno(r, "Failed to acquire shadow entry for group %s, ignoring: %m", gr->gr_name); + incomplete = ERRNO_IS_PRIVILEGE(r); + } + } else { + r = -EUCLEAN; + incomplete = true; } r = nss_group_to_group_record(gr, r >= 0 ? &sgrp : NULL, ret); @@ -948,6 +962,8 @@ int groupdb_iterator_get(UserDBIterator *iterator, GroupRecord **ret) { if (ret) (*ret)->incomplete = incomplete; + + iterator->n_found++; return r; } @@ -971,11 +987,11 @@ int groupdb_iterator_get(UserDBIterator *iterator, GroupRecord **ret) { iterator->n_found++; return synthetic_nobody_group_build(ret); } - } - /* if we found at least one entry, then ignore errors and indicate that we reached the end */ - if (r < 0 && iterator->n_found > 0) - return -ESRCH; + /* if we found at least one entry, then ignore errors and indicate that we reached the end */ + if (iterator->n_found > 0) + return -ESRCH; + } return r; } @@ -995,7 +1011,7 @@ int membershipdb_by_user(const char *name, UserDBFlags flags, UserDBIterator **r if (r < 0) return r; - iterator = userdb_iterator_new(LOOKUP_MEMBERSHIP); + iterator = userdb_iterator_new(LOOKUP_MEMBERSHIP, flags); if (!iterator) return -ENOMEM; @@ -1038,7 +1054,7 @@ int membershipdb_by_group(const char *name, UserDBFlags flags, UserDBIterator ** if (r < 0) return r; - iterator = userdb_iterator_new(LOOKUP_MEMBERSHIP); + iterator = userdb_iterator_new(LOOKUP_MEMBERSHIP, flags); if (!iterator) return -ENOMEM; @@ -1079,7 +1095,7 @@ int membershipdb_all(UserDBFlags flags, UserDBIterator **ret) { assert(ret); - iterator = userdb_iterator_new(LOOKUP_MEMBERSHIP); + iterator = userdb_iterator_new(LOOKUP_MEMBERSHIP, flags); if (!iterator) return -ENOMEM; @@ -1113,7 +1129,7 @@ int membershipdb_iterator_get( assert(iterator); for (;;) { - /* If we are iteratring through NSS acquire a new group entry if we haven't acquired one yet. */ + /* If we are iterating through NSS acquire a new group entry if we haven't acquired one yet. */ if (!iterator->members_of_group) { struct group *g;