Page MenuHomeFreeBSD

nscd: Warn when invalid configuration options are selected
ClosedPublic

Authored by allanjude on Jul 11 2020, 6:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 23 2024, 10:30 AM
Unknown Object (File)
Jan 14 2024, 4:12 AM
Unknown Object (File)
Dec 24 2023, 5:59 PM
Unknown Object (File)
Dec 20 2023, 3:22 AM
Unknown Object (File)
Dec 3 2023, 5:29 PM
Unknown Object (File)
Dec 3 2023, 5:29 PM
Unknown Object (File)
Dec 3 2023, 5:29 PM
Unknown Object (File)
Dec 3 2023, 5:29 PM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 32259
Build 29749: arc lint + arc unit

Event Timeline

markj added inline comments.
usr.sbin/nscd/parser.c
538–547

It is a bit weird to test what is effectively a bool using > 0. != 0 would be clearer, or just write if (invalid_value) {.

539

Missing a space for the continuing line.

This revision is now accepted and ready to land.Jul 11 2020, 6:57 PM

Agreed, I would have used a bool rather than an int, but this is an improvement regardless.

allanjude added inline comments.
usr.sbin/nscd/parser.c
538–547

right.

539

This is not actually a continuing line, LOG_ERR_2() is a macro that turns into syslog(LOG_ERR, "nscd error (from %s): %s", sender, ferror);
where the first line is 'sender' and the 2nd line is 'ferror'. Only the 2nd parameter has printf expansion.

allanjude marked 2 inline comments as done.

Address review feedback

This revision now requires review to proceed.Jul 11 2020, 7:41 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 11 2020, 7:44 PM
This revision was automatically updated to reflect the committed changes.