<div dir="ltr"><div><div>I didn't mean any present incorrect behaviour, but rather a weakness which may<br>(or may not) reappear at some point later and cause painful bugs (a zeroed-out<br>struct stands out way more than a gibberish-filled uninitialized one).<br>`do_filter` is two frames deeper from that `filter_result`'s declaration - and<br>that gives opportunity to use it before it's ready.<br><br>Personally I'd lose the `filter_result.src_prefix = NULL;`in xroute.c:290 <br>and leave the `= {0};` alone.<br><br></div>Thanks,<br></div>Stanislaw<br></div><br><div class="gmail_quote"><div dir="ltr">pt., 9 mar 2018 o 14:38 użytkownik Juliusz Chroboczek <<a href="mailto:jch@irif.fr">jch@irif.fr</a>> napisał:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>> I noticed that there's this new commit (b8fb6d896a234eaa06) which removes<br>
>> explicit initialization from check_xroutes() in xroute.c.<br>
<br>
> Fixed, thanks.<br>
<br>
Er, no.  do_filter unconditionally initialises the result, so the code is<br>
correct.  Am I missing something?<br>
<br>
-- Juliusz<br>
</blockquote></div>