Page MenuHomeFreeBSD

sendmail: do not use IN_CLASS macros
AbandonedPublic

Authored by karels on Oct 28 2021, 4:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 1 2024, 7:54 AM
Unknown Object (File)
Dec 23 2023, 1:27 AM
Unknown Object (File)
Jul 7 2023, 8:37 AM
Unknown Object (File)
Jun 11 2023, 6:49 PM
Unknown Object (File)
Apr 23 2023, 11:42 AM
Unknown Object (File)
Mar 5 2023, 7:51 AM
Unknown Object (File)
Feb 16 2023, 11:50 AM
Unknown Object (File)
Jan 9 2023, 6:04 AM
Subscribers

Details

Reviewers
gshapiro
Summary

sendmail was disecting addresses to see if they were on the loopback
"net". Instead, use IN_LOOPBACK(). This is more future-proof.

Test Plan

no functional change

Diff Detail

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

Event Timeline

If the current code works, I'd prefer to put this change in the upstream first and import it into the FreeBSD source on the next sendmail import. If this is causing breakage now, then we can proceed.

The current code works for now, but there is a proposal for a change that would make the old code fail. Note that IN_LOOPBACK() is not in Linux; not sure about other BSDs. The change could be conditional on #ifdef IN_LOOPBACK.

The change (with some minor tweaks) has been committed to the upstream open source distribution and will be part of the next vendor import into FreeBSD.

@karels: An associated release note entry was added, let me know if you would like the attribution modified:

Portability:
        Make use of IN_LOOPBACK, if defined, to determine if
        using a loopback address.  Patch from Mike Karels of
        FreeBSD.

Closing review, patch in progress upstream.