<div dir="ltr"><div dir="ltr">On Fri, Feb 5, 2021, at 4:04 PM Hal Murray via devel <<a href="mailto:devel@ntpsec.org">devel@ntpsec.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> What were you looking for in the branch?<br>
<br>
The code that would get the correct source address on server replies.<br></blockquote><div><br></div><div>The short version is I looked and it is not there^d^d^d^d^d anywhere I can see it.</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">
I don't even know if it is possible.  The current  code turns on the right <br>
option and then gets the source address when it reads a packet using recvmsg() <br>
or recvfrom().<br>
<br>
Is there a similar option to get the dest address?  Is there an option on send <br>
to specify the source address?<br></blockquote><div><br></div><div>It would appear yes, at least on StackOverflow.[1]</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">
How much will that slow things down?  Would we be willing to add the extra <br>
sockets to get that much of a speedup?  How ugly will that code be compared to <br>
the extra sockets?<br></blockquote><div><br></div><div>I don't know, I've been told to walk away.</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">
Can we rewrite the isc code that does all the work of keeping track of <br>
interfaces so that it is simpleer/smaller?<br></blockquote><div><br></div><div>Almost certainly. I had a tin dog program that I copied that spits out a list of interfaces. It'd need bulking out but I'm off.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> I tested it using ntpdig on a couple of machines running Kubuntu 20.04 and<br>
> one w/ macOS High Sierra. Nothing complicated though, I can't be bothered to<br>
> think of and set up cases. <br>
<br>
If you aren't willing to test your changes on interesting cases, then please don't work on this area.  Yes, the current code is ugly, but there is a reason for that.</blockquote><div><br></div><div> Okay.</div><div><br></div><div>[1] <a href="https://stackoverflow.com/a/5281706/9831509">https://stackoverflow.com/a/5281706/9831509</a></div></div></div>