Page MenuHomeFreeBSD

nscd step3
AcceptedPublic

Authored by david_crossfamilyweb.com on Wed, Oct 16, 2:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 2:22 PM
Unknown Object (File)
Fri, Nov 8, 2:14 PM
Unknown Object (File)
Tue, Nov 5, 3:50 PM
Unknown Object (File)
Sun, Oct 20, 11:40 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.Thu, Oct 17, 2:20 PM