<p>Matthew Fredrickson <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6173">View Change</a></p><p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Joshua,</p><p style="white-space: pre-wrap; word-wrap: break-word;">In order to use netsock2, we would have had to change the address<br>structure to ast_sockaddr but as the original dundi implementation<br>that used ipv4 did not adopt it we decided not to change it. We<br>were trying to minimize the amount of code change, whilst adding<br>the ipv6 functionality to minimise the chance of introducing new<br>bugs to dundi.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for your original two questions:<br>Q) "Is it possible for an IPv4 host to receive an IPv6 address? If<br>so for older versions what occurs?"</p><p style="white-space: pre-wrap; word-wrap: break-word;">A) We copied the way IAX2 implemented the IPv6 interface - in this<br>scenario we would not expect two Asterisk servers that are<br>configured to talk together with DUNDi to be running a mixed fleet<br>of IPv4 and IPv6, you would simply choose one and make sure your<br>entire fleet of Asterisk servers are running the same version - as<br>a result we don't see any issue with it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Q) "I don't think this works with IPv6 due to the scanning for :<br>which is present in IPv6 addresses"</p><p style="white-space: pre-wrap; word-wrap: break-word;">A) We have not seen any issues with this implementation in more<br>than 6 months of trials with a significant number of servers. We<br>will remove the TODO statement as we beleive is not needed.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Have submitted a new patch set that removes this TODO statement.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Kind regards,</p><p style="white-space: pre-wrap; word-wrap: break-word;">Adam</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hey Adam,</p><p style="white-space: pre-wrap; word-wrap: break-word;">A few of the devs on IRC had a discussion, and came to the conclusion that in order to move forward with this code, the netsock2 conversion is a requirement.  Also, since there are no tests accompanying this change, it will need to be against the master branch as well.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Best wishes,<br>Matthew Fredrickson</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/6173/3/pbx/pbx_dundi.c">File pbx/pbx_dundi.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/6173/3/pbx/pbx_dundi.c@1323">Patch Set #3, Line 1323:</a> <code style="font-family:monospace,monospace">                       memcpy(&trans->addr.i6, &p->addr.i6, sizeof(struct sockaddr_in6));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Is this a bug?  Memcpy struct differences between ipv4 and ipv6.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6173">change 6173</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6173"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ifbe298afc6416ba400db7be404a25994ad23968a </div>
<div style="display:none"> Gerrit-Change-Number: 6173 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Adam Secombe <adam.j.secombe@boeing.com> </div>
<div style="display:none"> Gerrit-Reviewer: Adam Secombe <adam.j.secombe@boeing.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 18 Dec 2017 19:32:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>