Date: Wed, 19 Jan 2022 14:23:51 GMT From: Cy Schubert <cy@FreeBSD.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org Subject: git: 931aa7a879bc - stable/12 - ipfilter: Fixup comment Message-ID: <202201191423.20JENpYT010687@gitrepo.freebsd.org>
next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by cy: URL: https://cgit.FreeBSD.org/src/commit/?id=931aa7a879bcb40d4e02c1d4fffef0a25b9319cc commit 931aa7a879bcb40d4e02c1d4fffef0a25b9319cc Author: Cy Schubert <cy@FreeBSD.org> AuthorDate: 2022-01-12 21:03:46 +0000 Commit: Cy Schubert <cy@FreeBSD.org> CommitDate: 2022-01-19 14:23:16 +0000 ipfilter: Fixup comment Fix comment documenting checksum block in ip_nat.c. Fastforward doesn't perform checksum. (cherry picked from commit 896a0153190937e071a31c682c6cf55e4c599886) --- sys/contrib/ipfilter/netinet/ip_nat.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/sys/contrib/ipfilter/netinet/ip_nat.c b/sys/contrib/ipfilter/netinet/ip_nat.c index 76a2dba15402..e990658551b3 100644 --- a/sys/contrib/ipfilter/netinet/ip_nat.c +++ b/sys/contrib/ipfilter/netinet/ip_nat.c @@ -5128,10 +5128,8 @@ ipf_nat_out(fin, nat, natadd, nflags) defined(BRIDGE_IPF) || defined(__FreeBSD__) else { /* - * Strictly speaking, this isn't necessary on BSD - * kernels because they do checksum calculation after - * this code has run BUT if ipfilter is being used - * to do NAT as a bridge, that code doesn't exist. + * We always do this on FreeBSD because this code doesn't + * exist in fastforward. */ switch (nat->nat_dir) {
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202201191423.20JENpYT010687>