From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [IPV6]: ROUTE:read ifname in rt6_info_route() Date: Thu, 16 Apr 2009 08:30:31 -0700 Message-ID: <20090416083031.1cc24489@nehalam> References: <49E710E1.3060209@miraclelinux.com> <20090416.043800.171462037.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: ysugiura@miraclelinux.com, netdev@vger.kernel.org To: David Miller Return-path: Received: from mail.vyatta.com ([76.74.103.46]:40161 "EHLO mail.vyatta.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752396AbZDPPah (ORCPT ); Thu, 16 Apr 2009 11:30:37 -0400 In-Reply-To: <20090416.043800.171462037.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 16 Apr 2009 04:38:00 -0700 (PDT) David Miller wrote: > From: Yuta sugiura > Date: Thu, 16 Apr 2009 20:05:05 +0900 > > > Hi, > > I have a suggestion about /proc/net/IPv6_route format. > > In rt6_info_route(), interface name is treated as a maximum 8 > > characters. > > But the type of (struct rt6_inf *)->u.dst.dev->name is char[IFNAMESIZ= > > 16] > > so, I think that it should change "%8s" to "%16s". > > Isn't there a way to pass the field length as a paramenter > to formatting functions? Then we can just pass in > IFNAMESIZE as that parameter. seq_printf(m, " %08x %08x %08x %08x %*s\n" rt->rt6i_metric, atomic_read(&rt->u.dst.__refcnt), rt->u.dst.__use, rt->rt6i_flags, IFNAMESIZE, rt->rt6i_dev ? rt->rt6i_dev->name : "")