aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKristof Provost <kp@FreeBSD.org>2021-07-14 10:17:03 +0000
committerKristof Provost <kp@FreeBSD.org>2021-08-07 07:09:35 +0000
commit44c47bc6d61ea295c6bb955a40f32c93a589f3ea (patch)
treeded80abb2c7d4648bc1a53118c5bb267ea7e95af
parent987a238d6eaaeb0d11c163135e4e1b4617153d2c (diff)
pf: locally originating connections with 'route-to' fail
Similar to the REPLY_TO shortcut (6d786845cf) we also can't shortcut ROUTE_TO. If we do we will fail to apply transformations or update the state, which can lead to premature termination of the connections. PR: 257106 MFC after: 3 weeks Sponsored by: Rubicon Communications, LLC ("Netgate") Differential Revision: https://reviews.freebsd.org/D31177 (cherry picked from commit 2c0d115bbc8f8ee3f011a5c4a69bcbf58c4b721f)
-rw-r--r--sys/netpfil/pf/pf.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/sys/netpfil/pf/pf.c b/sys/netpfil/pf/pf.c
index caae2b92f0d6..a7b429120c9e 100644
--- a/sys/netpfil/pf/pf.c
+++ b/sys/netpfil/pf/pf.c
@@ -338,12 +338,6 @@ VNET_DEFINE(struct pf_limit, pf_limits[PF_LIMIT_MAX]);
return (PF_DROP); \
if (PACKET_LOOPED(pd)) \
return (PF_PASS); \
- if ((d) == PF_OUT && \
- (s)->rule.ptr->rt == PF_ROUTETO && \
- (s)->rule.ptr->direction == PF_OUT && \
- (s)->rt_kif != NULL && \
- (s)->rt_kif != (i)) \
- return (PF_PASS); \
} while (0)
#define BOUND_IFACE(r, k) \