<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Mar 15, 2015 at 1:07 PM, Zbigniew Jędrzejewski-Szmek <span dir="ltr"><<a href="mailto:zbyszek@in.waw.pl" target="_blank">zbyszek@in.waw.pl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div>On Wed, Mar 11, 2015 at 08:13:47AM -0700, Shawn Landden wrote:<br>
> if we are going to have a function to fix up the deficiencies of<br>
> inet_pton(), better go all the way.<br>
> ---<br>
>  src/shared/in-addr-util.c | 17 +++++++++++++++++<br>
>  src/shared/in-addr-util.h |  1 +<br>
>  2 files changed, 18 insertions(+)<br>
><br>
> diff --git a/src/shared/in-addr-util.c b/src/shared/in-addr-util.c<br>
> index ade4012..b7532a6 100644<br>
> --- a/src/shared/in-addr-util.c<br>
> +++ b/src/shared/in-addr-util.c<br>
> @@ -206,7 +206,18 @@ int in_addr_to_string(int family, const union in_addr_union *u, char **ret) {<br>
>          return 0;<br>
>  }<br>
><br>
> +struct in6_addr in_addr_to_in6_addr(struct in_addr ipv4) {<br>
> +        struct in6_addr r = { { {<br>
> +                0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0xFF, 0xFF, 0, 0, 0, 0<br>
> +        } } };<br>
> +<br>
> +        r.s6_addr32[3] = ipv4.s_addr;<br>
> +<br>
> +        return r;<br>
> +}<br>
> +<br>
>  int in_addr_from_string(int family, const char *s, void *ret) {<br>
> +        struct in_addr v4mapped;<br>
><br>
>          assert(s);<br>
>          assert(ret);<br>
> @@ -214,6 +225,12 @@ int in_addr_from_string(int family, const char *s, void *ret) {<br>
>          if (!IN_SET(family, AF_INET, AF_INET6))<br>
>                  return -EAFNOSUPPORT;<br>
><br>
> +        if (family == AF_INET6)<br>
> +                if (inet_pton(AF_INET, s, &v4mapped) == 1) {<br>
> +                        *((struct in6_addr *)ret) = in_addr_to_in6_addr(v4mapped);<br>
> +                        return 0;<br>
> +                }<br>
> +<br>
</div></div>Hm, IIUC, an ipv6 address was specified, but the user gave an ipv4 address.<br>
Automatically converting seems like a way to entice confusion...<br>
<br></blockquote><div>Except that the linux kernel supports binding to ipv4 this way, (see ipv6(7)).</div><div><br></div><div>In the inet_pton man page:</div><div><br></div><div><div>BUGS</div><div>       AF_INET6 does not recognize IPv4 addresses.  An explicit IPv4-mapped IPv6 address must be supplied in src instead.</div></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Zbyszek<br>
<span><br>
>          errno = 0;<br>
>          if (inet_pton(family, s, ret) <= 0)<br>
>                  return errno ? -errno : -EINVAL;<br>
> diff --git a/src/shared/in-addr-util.h b/src/shared/in-addr-util.h<br>
> index 8b3a07c..f140ec9 100644<br>
> --- a/src/shared/in-addr-util.h<br>
> +++ b/src/shared/in-addr-util.h<br>
> @@ -37,6 +37,7 @@ int in_addr_equal(int family, const union in_addr_union *a, const union in_addr_<br>
>  int in_addr_prefix_intersect(int family, const union in_addr_union *a, unsigned aprefixlen, const union in_addr_union *b, unsigned bprefixlen);<br>
>  int in_addr_prefix_next(int family, union in_addr_union *u, unsigned prefixlen);<br>
>  int in_addr_to_string(int family, const union in_addr_union *u, char **ret);<br>
> +struct in6_addr in_addr_to_in6_addr(struct in_addr ipv4) _const_;<br>
>  int in_addr_from_string(int family, const char *s, void *ret);<br>
>  int in_addr_from_string_auto(const char *s, int *family, union in_addr_union *ret);<br>
>  unsigned char in_addr_netmask_to_prefixlen(const struct in_addr *addr);<br>
> --<br>
> 2.2.1.209.g41e5f3a<br>
><br>
</span>> _______________________________________________<br>
> systemd-devel mailing list<br>
> <a href="mailto:systemd-devel@lists.freedesktop.org" target="_blank">systemd-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
_______________________________________________<br>
systemd-devel mailing list<br>
<a href="mailto:systemd-devel@lists.freedesktop.org" target="_blank">systemd-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Shawn Landden</div>
</div></div>