Messages in this thread Patch in this message | | | From | "Jason A. Donenfeld" <> | Subject | [PATCH] vsprintf: do not append unset Scope ID to IPv6 | Date | Wed, 3 Feb 2016 11:41:58 +0100 |
| |
The sockaddr_in6 has the sin6_scope_id parameter. This contains the netdev index of the output device. When set to 0, sin6_scope_id is considered to be "unset" -- it has no Scope ID (see RFC4007). When it is set to >0, it has a Scope ID.
Other parts of the kernel respect htis. For example, code like this is sometimes used to populate a sockaddr_in6 from a skb:
sin6->sin6_scope_id = ipv6_iface_scope_id(ipv6_hdr(skb)->saddr, skb->skb_iif);
Such a code example makes a call to the ipv6_iface_scope_id function which is defined as follows (in ipv6.h):
static inline __u32 ipv6_iface_scope_id(const struct in6_addr *addr, int iface) { return __ipv6_addr_needs_scope_id(__ipv6_addr_type(addr)) ? iface : 0; }
So, here, either it has Scope ID, in which case it's returned, or it doesn't need a Scope ID, in which case 0 - "no scope id" - is returned.
Therefore, vsprintf should treat the 0 Scope ID the same way the rest of the v6 stack and the RFC treats it: 0 Scope ID means no Scope ID. And if there is no Scope ID, there is nothing to be printed.
So, this patch only prints the Scope ID when one exists, and does not print a Scope ID when one does not exist.
It turns out, too, that this is what developers want. The most common purpose of %pISpfsc is to show "what is in that sockaddr so that I can have some idea of how to connect to it?"
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> --- lib/vsprintf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/vsprintf.c b/lib/vsprintf.c index 48ff9c3..1b1b1c8 100644 --- a/lib/vsprintf.c +++ b/lib/vsprintf.c @@ -1194,7 +1194,7 @@ char *ip6_addr_string_sa(char *buf, char *end, const struct sockaddr_in6 *sa, p = number(p, pend, ntohl(sa->sin6_flowinfo & IPV6_FLOWINFO_MASK), spec); } - if (have_s) { + if (have_s && sa->sin6_scope_id) { *p++ = '%'; p = number(p, pend, sa->sin6_scope_id, spec); } -- 2.7.0
| |