<div dir="ltr"><div>Updated the original patch to just jump to cleanup, instead of rearranging things.</div><div><br></div><div>Also, polished up the proof of concept xgetaddrinfo (this needs a better name)</div><div>patch, and used it to implement -4, -6 (plus another patch to do -u).</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 29, 2017 at 6:59 PM, enh <span dir="ltr"><<a href="mailto:enh@google.com" target="_blank">enh@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ping? various networking folks are looking to add tests that use<br>
netcat, and i'd rather start them off on toybox netcat rather than BSD<br>
and then have to move them across to toybox later. (obviously there's<br>
other missing stuff in toybox, but these patches are the only things<br>
they actually need right now.)<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Jun 28, 2017 at 3:47 PM, Josh Gao <<a href="mailto:jmgao@google.com">jmgao@google.com</a>> wrote:<br>
> On Sun, Jun 25, 2017 at 12:14 PM, Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br>
>><br>
>> 1) switching it to use xconnect() which it predates, and which is hard<br>
>> because the various users in tree all want slighty different things out<br>
>> of the getaddrinfo() plumbing and I've made a couple attempts to<br>
>> unify/genericize it but keep getting pulled alway by $DAYJOB crisis du<br>
>> jour halfway through and forgetting what design problem details I was<br>
>> halfway through solving and have to start over again...<br>
><br>
><br>
> BTW, I took a quick look at this because we have users that want -4/-6 (and<br>
> IPv6 support in general). `nc -s` makes it so that you can't use xconnect<br>
> because you don't know what to bind to until after you've resolved the<br>
> target<br>
> address. Something like xbind_and_connect might work, but there's also<br>
> things<br>
> that we might want to do in between socket and bind (e.g. setting<br>
> SO_REUSEADDR).<br>
><br>
> The thing that everyone really wants is a way to iterate over getaddrinfo<br>
> results;<br>
> maybe that's what should be exposed? I have a rough proof of concept patch<br>
> attached that implements this and uses it in netcat.<br>
><br>
> (There's also another edge case with -s: what happens if the host you pass<br>
> in<br>
> resolves to multiple addresses? OpenBSD's netcat seems to just bind the<br>
> first<br>
> compatible address it resolves to, so we can maybe just ignore this.)<br>
><br>
> -Josh<br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> ______________________________<wbr>_________________<br>
> Toybox mailing list<br>
> <a href="mailto:Toybox@lists.landley.net">Toybox@lists.landley.net</a><br>
> <a href="http://lists.landley.net/listinfo.cgi/toybox-landley.net" rel="noreferrer" target="_blank">http://lists.landley.net/<wbr>listinfo.cgi/toybox-landley.<wbr>net</a><br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Elliott Hughes - <a href="http://who/enh" rel="noreferrer" target="_blank">http://who/enh</a> - <a href="http://jessies.org/~enh/" rel="noreferrer" target="_blank">http://jessies.org/~enh/</a><br>
Android native code/tools questions? Mail me/drop by/add me as a reviewer.<br>
</font></span></blockquote></div><br></div>