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

Reply via email to