2024-03-09 08:24:56

by Christophe JAILLET

Subject: [PATCH] pstore/zone: Don't clear memory twice

There is no need to call memset(..., 0, ...) on memory allocated by
kcalloc(). It is already zeroed.

Remove the redundant call.

Signed-off-by: Christophe JAILLET <[email protected]>
---
fs/pstore/zone.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/fs/pstore/zone.c b/fs/pstore/zone.c
index abca117725c8..694db616663f 100644
--- a/fs/pstore/zone.c
+++ b/fs/pstore/zone.c
@@ -1217,7 +1217,6 @@ static struct pstore_zone **psz_init_zones(enum pstore_type_id type,
pr_err("allocate for zones %s failed\n", name);
return ERR_PTR(-ENOMEM);
}
- memset(zones, 0, c * sizeof(*zones));

for (i = 0; i < c; i++) {
zone = psz_init_zone(type, off, record_size);
--
2.44.0

2024-03-09 20:34:06

by Kees Cook

Subject: Re: [PATCH] pstore/zone: Don't clear memory twice

On Sat, 09 Mar 2024 09:24:27 +0100, Christophe JAILLET wrote:
> There is no need to call memset(..., 0, ...) on memory allocated by
> kcalloc(). It is already zeroed.
>
> Remove the redundant call.
>
>

Applied to for-next/pstore, thanks!

[1/1] pstore/zone: Don't clear memory twice
https://git.kernel.org/kees/c/c8d25d696f52

Take care,

--
Kees Cook