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
| ||
|
Date: Tue, 27 Sep 2016 19:16:04 +0900 From: YOSHIFUJI Hideaki <hideaki.yoshifuji@...aclelinux.com> To: Maciej Żenczykowski <zenczykowski@...il.com>, Maciej Żenczykowski <maze@...gle.com>, "David S . Miller" <davem@...emloft.net> Cc: hideaki.yoshifuji@...aclelinux.com, netdev@...r.kernel.org, Erik Kline <ek@...gle.com>, Lorenzo Colitti <lorenzo@...gle.com>, Hannes Frederic Sowa <hannes@...essinduktion.org> Subject: Re: [PATCH v5 1/7] ipv6 addrconf: enable use of proc_dointvec_minmax in addrconf_sysctl Hi, Maciej Żenczykowski wrote: > From: Maciej Żenczykowski <maze@...gle.com> > > Signed-off-by: Maciej Żenczykowski <maze@...gle.com> > --- > net/ipv6/addrconf.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c > index 2f1f5d439788..11fa1a5564d4 100644 > --- a/net/ipv6/addrconf.c > +++ b/net/ipv6/addrconf.c > @@ -6044,8 +6044,14 @@ static int __addrconf_sysctl_register(struct net *net, char *dev_name, > > for (i = 0; table[i].data; i++) { > table[i].data += (char *)p - (char *)&ipv6_devconf; > - table[i].extra1 = idev; /* embedded; no ref */ > - table[i].extra2 = net; > + /* If one of these is already set, then it is not safe to > + * overwrite either of them: this makes proc_dointvec_minmax > + * usable. > + */ > + if (!table[i].extra1 && !table[i].extra2) { > + table[i].extra1 = idev; /* embedded; no ref */ > + table[i].extra2 = net; > + } > } > > snprintf(path, sizeof(path), "net/ipv6/conf/%s", dev_name); > This seems nothing to do with the RFC7559 changes. Why don't you submit this as a separate patch? -- Hideaki Yoshifuji <hideaki.yoshifuji@...aclelinux.com> Technical Division, MIRACLE LINUX CORPORATION
Powered by blists - more mailing lists