diff options
| author | Henning Brauer <henning@cvs.openbsd.org> | 2007-08-28 16:09:13 +0000 |
|---|---|---|
| committer | Henning Brauer <henning@cvs.openbsd.org> | 2007-08-28 16:09:13 +0000 |
| commit | 4d95d84eff03832c48d58dec3beee809f796c7a9 (patch) | |
| tree | 09d24371ef795519cf205477696389392142c780 | |
| parent | b8733342a874cb8dd010675510eb731bcb132401 (diff) | |
showing this diff is shameful...
if (r != NULL && r->rtableid >= 0)
- m->m_pkthdr.pf.rtableid = m->m_pkthdr.pf.rtableid;
+ m->m_pkthdr.pf.rtableid = r->rtableid;
fortunately it is in pf_send_tcp and thus the effect is very limited, RSTs
sent due to "block return(-rst)" could be routed using the main routing
table instead of an alternate one specified on the block rule.
spotted by Janjaap van Velthooven <janjaap@stack.nl>
| -rw-r--r-- | sys/net/pf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net/pf.c b/sys/net/pf.c index 5869247f521..7518038a2e8 100644 --- a/sys/net/pf.c +++ b/sys/net/pf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf.c,v 1.553 2007/08/23 11:15:49 dhartmei Exp $ */ +/* $OpenBSD: pf.c,v 1.554 2007/08/28 16:09:12 henning Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -1598,7 +1598,7 @@ pf_send_tcp(const struct pf_rule *r, sa_family_t af, m->m_pkthdr.pf.tag = rtag; if (r != NULL && r->rtableid >= 0) - m->m_pkthdr.pf.rtableid = m->m_pkthdr.pf.rtableid; + m->m_pkthdr.pf.rtableid = r->rtableid; #ifdef ALTQ if (r != NULL && r->qid) { |