On Thu, 24 Sep 2020 at 22:37, Paul Wouters <[email protected]> wrote:

> On Thu, 24 Sep 2020, Andrew Cagney wrote:
>
> > Once ddns resolves a name it updates .host_addr (from AF_UNSPEC to a
> valid address).
> > This, in turn, can update .client to .host_addr/32.
> > And updating .client changes .maskbits from 0 to 32, say.
> > Since c->prio is based on .maskbits, should it too be updated?
>
> Ahh yes it should.
>
>
>
but things are starting to look weird.  Since this happens before orient():

c->policy_prio = (((policy_prio_t)c->spd.this.client.maskbits << 17) |
 ((policy_prio_t)c->spd.that.client.maskbits << 8) |
 ((policy_prio_t)1));
}

.this and .that could end up being completely backardwards.
_______________________________________________
Swan-dev mailing list
[email protected]
https://lists.libreswan.org/mailman/listinfo/swan-dev

Reply via email to