Re: [RFC PATCH 1/3] net: Remove unnecessary memset in __snmp6_fill_stats64
From: Florian Westphal <fw@strlen.de>
Date: 2016-08-08 11:12:38
Also in:
lkml
Jia He [off-list ref] wrote:
quoted hunk ↗ jump to hunk
buff[] will be assigned later, so memset is not necessary. Signed-off-by: Jia He <redacted> Cc: "David S. Miller" <davem@davemloft.net> Cc: Alexey Kuznetsov <redacted> Cc: James Morris <jmorris@namei.org> Cc: Hideaki YOSHIFUJI <redacted> Cc: Patrick McHardy <redacted> --- net/ipv6/addrconf.c | 1 - 1 file changed, 1 deletion(-)diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index ab3e796..43fa8d0 100644 --- a/net/ipv6/addrconf.c +++ b/net/ipv6/addrconf.c@@ -4967,7 +4967,6 @@ static inline void __snmp6_fill_stats64(u64 *stats, void __percpu *mib, BUG_ON(pad < 0); - memset(buff, 0, sizeof(buff)); buff[0] = IPSTATS_MIB_MAX; for_each_possible_cpu(c) {
for (i = 1; i < IPSTATS_MIB_MAX; i++)
buff[i] += snmp_get_cpu_field64(mib, c, i, syncpoff);
Without memset result of buff[i] += ... is undefined.