<p>Ok, if you are saying your configuration fix would correctly work to check the address type at run-time, then I guess it would work. Otherwise, it would be all IPv6 or all IPv4.</p>
<p>The fix you added that adds the brackets automatically at run-time for $siz is the best way actually, as then you do not need cryptic configuration files.</p>
<p>So, in any case, thanks very much for adding the new function that automatically adds the brackets when needed.</p>
<p>If I have permission from Verizon, I might like to contribute in the future. I can see a need to immediately parse the incoming messages and immediately reject any message with any part not formed correctly, rather than passing bad things through the code and crashing later. Also, would be good to switch to not use the old depricated system calls for networking (avoids dangerous mucking with the IP address structures and also avoid case switches for IPv4 vs IPv6). The new calls reduce the specific code for IPv4 vs IPv6 to an absolute minimum. I would  also use the system calls to convert IP address to string instead of the hand written ones, etc.</p>
<p>Don.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/issues/1136#issuecomment-310512913">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZRKGwfHduKEZKHFFuWlP-E8yZS4Rks5sGuIBgaJpZM4NkKgo">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZTjo8ZKuZ37Ubg646HK6NKFaLlktks5sGuIBgaJpZM4NkKgo.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/kamailio/kamailio/issues/1136#issuecomment-310512913"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@doncarr in #1136: Ok, if you are saying your configuration fix would correctly work to check the address type at run-time, then I guess it would work. Otherwise, it would be all IPv6 or all IPv4. \r\n\r\nThe fix you added that adds the brackets automatically at run-time for $siz is the best way actually, as then you do not need cryptic configuration files.\r\n\r\nSo, in any case, thanks very much for adding the new function that automatically adds the brackets when needed.\r\n\r\nIf I have permission from Verizon, I might like to contribute in the future. I can see a need to immediately parse the incoming messages and immediately reject any message with any part not formed correctly, rather than passing bad things through the code and crashing later. Also, would be good to switch to not use the old depricated system calls for networking (avoids dangerous mucking with the IP address structures and also avoid case switches for IPv4 vs IPv6). The new calls reduce the specific code for IPv4 vs IPv6 to an absolute minimum. I would  also use the system calls to convert IP address to string instead of the hand written ones, etc. \r\n\r\nDon."}],"action":{"name":"View Issue","url":"https://github.com/kamailio/kamailio/issues/1136#issuecomment-310512913"}}}</script>