[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