On Tue, Mar 29, 2016 at 6:35 AM, Timo Teräs <[email protected]> wrote: > Scope is the /proc bitmask of scope type, not the scope id. The > scope id should be set to the ifindex when the address is link > local. But for printing purposes we don't need to set it at all > since the line is already associated to interface by the header > line. > > Signed-off-by: Timo Teräs <[email protected]> > --- > Sorry, should've checked more thoroughly before the first patch > I was just confused with the local variable name and wanted to > get done with this. > > networking/interface.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/networking/interface.c b/networking/interface.c > index 90c1449..5db0cb7 100644 > --- a/networking/interface.c > +++ b/networking/interface.c > @@ -885,7 +885,7 @@ static void ife_print6(struct interface *ptr) > inet_pton(AF_INET6, addr6, > (struct sockaddr *) &sap.sin6_addr); > sap.sin6_family = AF_INET6; > - sap.sin6_scope_id = scope; > + sap.sin6_scope_id = 0;
There is "memset(&sap, 0, sizeof(sap))" a few lines up. Is it not enough? _______________________________________________ busybox mailing list [email protected] http://lists.busybox.net/mailman/listinfo/busybox
