Page MenuHomeFreeBSD

nscd step3
AcceptedPublic

Authored by david_crossfamilyweb.com on Oct 16 2024, 2:14 AM.
Tags
None
Referenced Files
F156467351: D47145.diff
Wed, May 13, 9:22 PM
Unknown Object (File)
Tue, May 12, 5:22 AM
Unknown Object (File)
Tue, May 12, 1:25 AM
Unknown Object (File)
Fri, May 1, 11:32 AM
Unknown Object (File)
Fri, May 1, 11:24 AM
Unknown Object (File)
Tue, Apr 28, 1:41 AM
Unknown Object (File)
Fri, Apr 24, 4:44 PM
Unknown Object (File)
Wed, Apr 15, 3:19 PM
Subscribers

Details

Reviewers
markj
Summary

Fixes for nscd.

  1. rename the symbol in nscd, not exported in Makefile yet, so not a visible change
  2. fix negative cache response for agent lookups

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

markj added inline comments.
usr.sbin/nscd/nscd.c
587

Thinking about this a bit more - suppose someone else has locally fixed the problem in nscd, where the symbol doesn't get exported. Now we're gratuitously making libc incompatible with their nscd. That is, if they upgrade libc, their nscd will stop working properly, but libc should always avoid breaking backwards compatibility.

The likelihood of this happening is small, so the change is unlikely to cause problems, but the advantage of the change is also very small.

I don't insist on dropping this change, but I don't really like it.

usr.sbin/nscd/query.c
809

It's not obvious to me that we want to make this change for the res == NS_RETURN case, but none of the existing lookup functions appear to return that, so ok.

This revision is now accepted and ready to land.Oct 17 2024, 2:20 PM