Date: Wed, 19 Jan 2022 14:21:29 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: d600d25179f2 - stable/13 - ipfilter: Fixup comment Message-ID: <202201191421.20JELTx8007205@gitrepo.freebsd.org>
next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by cy: URL: https://cgit.FreeBSD.org/src/commit/?id=d600d25179f23479c66ddfc3962b65c92f07dc2a commit d600d25179f23479c66ddfc3962b65c92f07dc2a 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:19:24 +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 cab4700be4a7..09430c1fdfe5 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?202201191421.20JELTx8007205>