<div dir="ltr">Hi,<div><br></div><div>Confirmed that the code I applied in the last patch is working fine. Also confirmed that the latest patch supplied works successfully with:</div><div><br></div><div>patch -p1 <š<span style="font-size:13px;font-family:arial,sans-serif">bird-1.3.11-match-T_IP-against-T_PREFIX_SET-v2.patch</span></div>
<div><span style="font-size:13px;font-family:arial,sans-serif"><br></span></div><div><span style="font-size:13px;font-family:arial,sans-serif">in top level directory of bird-1.3.11 source, and also confirmed that the function you suggested is working fine.</span></div>
<div><span style="font-size:13px;font-family:arial,sans-serif"><br></span></div><div><span style="font-size:13px;font-family:arial,sans-serif">Thank you very much!</span></div><div><span style="font-size:13px;font-family:arial,sans-serif"><br>
</span></div><div><span style="font-size:13px;font-family:arial,sans-serif">-JJ</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 11, 2013 at 3:08 AM, Sergey Popovich <span dir="ltr"><<a href="mailto:popovich_sergei@mail.ru" target="_blank">popovich_sergei@mail.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">÷ ÐÉÓØÍÅ ÏÔ 9 ÎÏÑÂÒÑ 2013 07:42:06 ÷Ù ÎÁÐÉÓÁÌÉ:<br>
<div class="im">> Hi,<br>
><br>
> I applied this code manually:<br>
><br>
> + šif ((v1.type == T_IP) && (v2.type == T_PREFIX_SET)) {<br>
> + š šv1.val.px.len = MAX_PREFIX_LENGTH;<br>
> + š šreturn trie_match_fprefix(v2.val.ti, &v1.val.px);<br>
> + š}<br>
><br>
> To filter.c<br>
><br>
> However maybe the patch failed because I don't have the following:<br>
><br>
> š šif (v2.type != T_SET)<br>
> š š šreturn CMP_ERROR;<br>
><br>
><br>
> Maybe the patch that was sent was within a different branch than the main<br>
> 1.3.11 that's available for download?<br>
><br>
<br>
<br>
</div>Oh sorry, John, I patch really was made on top of current upstream tree, not<br>
1.3.11. In attachment udpated patch version applied on top of 1.3.11 stable<br>
tree.<br>
<br>
Sorry for that.<br>
<br>
And yes, really according to documentation we could match only on set with<br>
elements of same tyme. On the other hand, an ip type may be easily represented<br>
as prefix with length of max AFI prefix length (32 - IPv4, 128 - IPv6, so I<br>
see no problem using this conversion of ip to prefix type.<br>
<br>
Moreover on IPv4 BIRD build there is implicit conversion of ip type to quad<br>
as on IPv4 they basically represent value with same format.<br>
<br>
Thanks for your note.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
SP5474-RIPE<br>
Sergey Popovich<br>
</font></span></blockquote></div><br></div>