Page MenuHomeFreeBSD

syslogd: Clean up allowedpeer structure
ClosedPublic

Authored by jfree on Aug 8 2023, 3:25 PM.
Tags
None
Referenced Files
F86618501: D41365.diff
Sun, Jun 23, 3:05 AM
F86599818: D41365.id125836.diff
Sat, Jun 22, 8:03 PM
Unknown Object (File)
May 19 2024, 4:34 PM
Unknown Object (File)
May 19 2024, 4:32 PM
Unknown Object (File)
May 19 2024, 4:32 PM
Unknown Object (File)
May 16 2024, 12:34 AM
Unknown Object (File)
May 15 2024, 10:13 PM
Unknown Object (File)
May 10 2024, 2:51 AM
Subscribers

Details

Summary
  • Use boolean values in validate() and struct allowedpeer.
  • Remove useless peer count loop.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jfree requested review of this revision.Aug 8 2023, 3:25 PM
markj added inline comments.
usr.sbin/syslogd/syslogd.c
206

By the way, we now use anonymous unions in quite a few places, so you could instead write

union {
    struct {
        struct sockaddr_storage a_addr;
        ...
};

and get rid of those horrible defines. But that should be done in a separate diff if at all.

This revision is now accepted and ready to land.Aug 9 2023, 7:24 PM
jfree edited the summary of this revision. (Show Details)

Update after rebase

This revision now requires review to proceed.Aug 10 2023, 11:08 PM
This revision is now accepted and ready to land.Aug 11 2023, 2:50 PM
This revision was automatically updated to reflect the committed changes.