<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 23, 2023 at 3:17 PM Alexander Zubkov <<a href="mailto:green@qrator.net">green@qrator.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 23, 2023 at 3:06 PM Ondrej Zajicek <<a href="mailto:santiago@crfreenet.org" target="_blank">santiago@crfreenet.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Jan 23, 2023 at 12:40:30AM +0100, Alexander Zubkov wrote:<br>
> Hi all,<br>
> <br>
> A quick try to fix the problem. But I'm not sure in complete correctness<br>
> though.<br>
<br>
Hi<br>
<br>
That looks more-or-less OK, will merge.<br>
<br>
> -    ipa_equal(x->addr, y->addr);<br>
> +    ipa_equal(x->addr, y->addr) &&<br>
> +    ipa_equal(x->addr2, y->addr2);<br>
<br>
I think undefined addr2 should work like wildcard, i.e. the condition should be:<br>
<br></blockquote><div><br></div><div>Maybe. I do not know well how this lock works. If different lock keys can affect another. And in this case it is probably better to fix "local" role for that second address and reflect it in its name.<br></div></div></div></blockquote><div><br></div><div>I think even better to call this wildcard_addr, for example. So if something else needs this wildcard feature, it is clear which addr to use in the lock object.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
 ipa_equal(x->addr, y->addr) &&<br>
 (ipa_zero(x->addr2) || ipa_zero(y->addr2) || ipa_equal(x->addr2, y->addr2));<br>
<br>
(Undefined local ip will be resolved to some ip and may collide with<br>
defined ones.)<br>
<br>
-- <br>
Elen sila lumenn' omentielvo<br>
<br>
Ondrej 'Santiago' Zajicek (email: <a href="mailto:santiago@crfreenet.org" target="_blank">santiago@crfreenet.org</a>)<br>
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, <a href="http://wwwkeys.pgp.net" rel="noreferrer" target="_blank">wwwkeys.pgp.net</a>)<br>
"To err is human -- to blame it on a computer is even more so."<br>
</blockquote></div></div>
</blockquote></div></div>