On Mon, Jan 04, 2021 at 03:26:15PM +0100, Alexandr Nedvedicky wrote:
> you refactoring diff requires a minor finishing touch to keep the
> stuff compiling:
Did I commit something that does not compile? I just made cvs
update on another machine. There it worked.
The rt_kif in pf_state still exists. The diff below should not
be necessary. Maybe you forgot to clean pfvar.h.
bluhm
> ----8<-------8<-------8<-------8<-------8<------------8<----
> diff --git a/sys/net/pf.c b/sys/net/pf.c
> index b8766df1686..3f9f5b13add 100644
> --- a/sys/net/pf.c
> +++ b/sys/net/pf.c
> @@ -3428,16 +3428,13 @@ pf_set_rt_ifp(struct pf_state *s, struct pf_addr
> *saddr, sa_family_t af,
> struct pf_rule *r = s->rule.ptr;
> int rv;
>
> - s->rt_kif = NULL;
> if (!r->rt)
> return (0);
>
> rv = pf_map_addr(af, r, saddr, &s->rt_addr, NULL, sns,
> &r->route, PF_SN_ROUTE);
> - if (rv == 0) {
> - s->rt_kif = r->route.kif;
> + if (rv == 0)
> s->natrule.ptr = r;
> - }
>
> return (rv);
> }
> ----8<-------8<-------8<-------8<-------8<------------8<----
>
>
> thanks and
> regards
> sashan