<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 19, 2017 at 2:57 PM, Ferry Huberts <span dir="ltr"><<a href="mailto:mailings@hupie.com" target="_blank">mailings@hupie.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
On 19/01/17 13:33, Henning Rogge wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
Hi,<br>
<br>
back from (two) business trips...<br>
<br>
On Mon, Jan 16, 2017 at 6:14 PM, Ferry Huberts <<a href="mailto:mailings@hupie.com" target="_blank">mailings@hupie.com</a><br></span><span class="">
<mailto:<a href="mailto:mailings@hupie.com" target="_blank">mailings@hupie.com</a>>> wrote:<br>
<br>
    I think I may have stumbled onto a MUCH easier fix to this problem.<br>
<br>
<br>
    The actual problem is quite involved (with fragmented hello messages<br>
    etc) but it all boils down to a later UNSPEC link overwriting a<br>
    previous SYM/ASYM/other link on the receiving end.<br>
<br>
    Since the neighbours in a hello message are already ordered when the<br>
    hello message is sent out, the following patch also solves it.<br>
<br>
    Opinions?<br>
<br>
<br>
I like the idea of solve the problem by sorting, but I worry that doing<br>
so on the TX side is a bit "brittle"... its easy to break because there<br>
is nothing on the TX side that needs this order. And there will always<br>
be older Olsrd installations on many community networks.<br>
<br>
How difficult would it be to do the sorting on the RX side<br>
in deserialize_hello() (process_package.c: 318..)?<br>
<br>
Instead of building them into a single linked list, have an array with<br>
MAX_LINK lists, one for each link type... and then concatenate this<br>
lists before parsing in the right order.<br>
<br>
This way we also fix the problem appearing again because of an outdated<br>
Olsrd sending bad data, right?<br>
<br>
</span></blockquote>
<br>
<br>
If we do that as a follow-up patch? Would that be ok?<br></blockquote><div><br></div><div>Sounds good... quick fix now to solve the problem for the current code, cleanup later to make it future proof.</div><div><br></div><div>Henning</div></div></div></div>