[Rpm-maint] [rpm-software-management/rpm] Add back support for NSS-based user/group resolution (PR #4085)
Panu Matilainen
notifications at github.com
Tue Jan 20 10:15:20 UTC 2026
@pmatilai commented on this pull request.
> long id;
- if (lookup_num(pwfile(), thisUname, 0, 2, &id))
- return -1;
- rpmug->unameMap.insert({thisUname, id});
+ if (pwfile() || !rstreq(rpmcliRootDir, "/")) {
You could probably also consolidate the chroot checking inside getpath(), so these checks become simply (pseudocode)
```
if (using_file())
... lookup from file ...
else
... lookup from nss ...
```
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/4085#discussion_r2707681582
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/4085/review/3681413213 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20260120/0eb2c8e8/attachment.htm>
More information about the Rpm-maint
mailing list