[sr-dev] [kamailio/kamailio] core: Added methods for ip conversation. (#1962)

Daniel-Constantin Mierla notifications at github.com
Wed May 22 10:33:54 CEST 2019


There are already functions to parse a string to ip struct for ipv4 and ipv4, see str2ip() and str2ip6() in `src/core/resolve.h `. In that struct you have the `af` field. In most of the code, the two functions were used one after the other to parse ip addresses, if str2ip() fails, then try str2ip6().

We can add a new function str2ipx() as a wrapper to this logic, but otherwise, I see no reason to add the new functions proposed by your pull request.

Also, the one for trimming `[ ]` for ipv6 with copy over is not that useful in a global scope from my point of view. Moreover, there are no safety checks over the size of the target pointer, nor comments requesting a minimum value for that. This can be in the module that needs it.

I am closing this PR without merge. If people want to discuss more, then keep adding comments and if it is decided, it can be reopened.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1962#issuecomment-494707973
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20190522/e81b5ee7/attachment-0001.html>


More information about the sr-dev mailing list