netdev - Re: [PATCH iproute2 0/2] lib names: Refactoring and cleanups

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20141210154652.60828ea4@urahara>
Date:	Wed, 10 Dec 2014 15:46:52 -0800
From:	Stephen Hemminger <stephen@...workplumber.org>
To:	vadim4j@...il.com
Cc:	netdev@...r.kernel.org
Subject: Re: [PATCH iproute2 0/2] lib names: Refactoring and cleanups

On Wed, 10 Dec 2014 17:10:29 +0200
vadim4j@...il.com wrote:

> On Tue, Dec 09, 2014 at 08:38:56PM -0800, Stephen Hemminger wrote:
> > On Sat,  6 Dec 2014 04:05:10 +0200
> > Vadim Kochan <vadim4j@...il.com> wrote:
> > 
> > > Some cleanups and refactoring in lib/rt_names.c:
> > > 
> > >     #1 Replaced using of /etc/iproute2 path by CONFDIR define
> > >         when initializing tables of group names.
> > > 
> > >     #2 Added helper to have one func for parsing id and names from
> > >         db files.
> > > 
> > > Vadim Kochan (2):
> > >   lib names: Use CONFDIR for specify 'group' file path
> > >   lib names: Add helper func for parse id and name from file
> > > 
> > >  lib/rt_names.c | 70 +++++++++++++++++++++++++++++++++-------------------------
> > >  1 file changed, 40 insertions(+), 30 deletions(-)
> > > 
> > 
> > Both applied
> 
> Hi Stephen,
> 
> I see a warning (after 'make clean && make') about using 'const' in lib/rt_names.c:
> 
>     static const char * rtnl_rtscope_tab[256] = {
>             "global",
>     };
> 
> which came with PATCH with a subject:
> 
>     "lib names: Add helper func for parse id and name from file" (f00073e8b)
> 
> but in the PATCH version which was sent by me in email I did not see this.
> 
> Why it was needed ?
> 
> Regards,
> Vadim

My gaff. Fixed but not pushed, holding of until include snafu is fixed
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html