Page MenuHomeFreeBSD

nscd step3
Needs ReviewPublic

Authored by david_crossfamilyweb.com on Oct 16 2024, 2:14 AM.
Tags
None
Referenced Files
F175632846: D47145.id189513.diff
Mon, Oct 12, 4:35 AM
Unknown Object (File)
Fri, Oct 2, 2:05 PM
Unknown Object (File)
Thu, Sep 24, 6:33 AM
Unknown Object (File)
Sep 5 2026, 1:00 AM
Unknown Object (File)
Aug 30 2026, 10:48 AM
Unknown Object (File)
Aug 29 2026, 9:00 AM
Unknown Object (File)
Aug 20 2026, 5:44 AM
Unknown Object (File)
Aug 15 2026, 11:02 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
This revision now requires review to proceed.Sun, Oct 11, 9:40 PM