netdev - Re: [iproute PATCH v5 3/5] Replace malloc && memset by calloc

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <abf02ad6-e20e-380d-8358-1c1d5c6a848c@cumulusnetworks.com>
Date:	Mon, 18 Jul 2016 10:10:09 -0600
From:	David Ahern <dsa@...ulusnetworks.com>
To:	Phil Sutter <phil@....cc>, Stephen Hemminger <shemming@...cade.com>
Cc:	Daniel Borkmann <daniel@...earbox.net>,
	Nicolas Dichtel <nicolas.dichtel@...nd.com>,
	Julien Floret <julien.floret@...nd.com>,
	David Laight <David.Laight@...LAB.COM>, netdev@...r.kernel.org
Subject: Re: [iproute PATCH v5 3/5] Replace malloc && memset by calloc

On 7/18/16 8:48 AM, Phil Sutter wrote:
> This only replaces occurrences where the newly allocated memory is
> cleared completely afterwards, as in other cases it is a theoretical
> performance hit although code would be cleaner this way.
>
> Signed-off-by: Phil Sutter <phil@....cc>
> ---
> Changes since v2:
> - Fix checkpatch errors.
> ---
>  genl/genl.c        |  3 +--
>  lib/names.c        |  7 ++-----
>  misc/lnstat.c      |  6 ++----
>  misc/lnstat_util.c |  4 +---
>  tc/em_canid.c      |  4 ++--
>  tc/m_action.c      |  3 +--
>  tc/m_ipt.c         | 13 ++++---------
>  tc/m_pedit.c       |  3 +--
>  tc/tc.c            |  9 +++------
>  tc/tc_bpf.c        |  4 +---
>  tc/tc_class.c      |  3 +--
>  tc/tc_exec.c       |  3 +--
>  12 files changed, 20 insertions(+), 42 deletions(-)

Acked-by: David Ahern <dsa@...ulusnetworks.com>