Skip to content

kzfree() v2

Johannes Weiner dm-devel linux-mm lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest7 patches 18 repliesIn review

v2 from the cover letter

- EXPORT_SYMBOL(kzfree), thanks linker
- remove superfluous NULL checks, thanks Pekka
- mention `security' in the description

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/7slab: introduce kzfree()include/linux/slab.h mm/util.c+21 -016 replies
2/7crypto: use kzfree()crypto/api.c+1 -41 reply
3/7s390: use kzfree()arch/s390/crypto/prng.c drivers/s390/crypto/zcrypt_pcixcc.c+2 -40 replies
4/7md: use kzfree()drivers/md/dm-crypt.c+2 -40 replies
5/7usb: use kzfree()drivers/usb/host/hwa-hc.c drivers/usb/wusbcore/cbaf.c+2 -41 reply
6/7cifs: use kzfree()fs/cifs/connect.c fs/cifs/misc.c+3 -130 replies
7/7ecryptfs: use kzfree()fs/ecryptfs/keystore.c fs/ecryptfs/messaging.c+2 -40 replies

Reviews trailers per patch

All 7 patches have a Reviewed-by or Acked-by.

PatchPekka EnbergTyler HicksSteve FrenchChristoph LameterDavid VrabelHerbert Xu
1/7R-b A-b
2/7R-b A-b
3/7R-b
4/7R-b
5/7R-b A-b
6/7R-b A-b
7/7R-b A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 6 reviewers.

Read the v1 thread

Cover letter [patch 0/7] kzfree() v2

This series introduces kzfree() and converts callsites which do memset() + kzfree() explicitely.

The caller intention is to ensure that security-sensitive data are cleared from slab objects before they are passed back to the allocator.

This also removes the last modular ksize() user (crypto/api.c) again by converting it to kzfree() which figures out the length of the memory region to zero internally.

I left out drivers/w1/w1{,_int}.c and dropped the conversion of drivers/atm/mpoa_caches.c in this iteration as I think they don't strictly need the zeroeing and the memsetting should probably be removed [ added Chas Williams and Evgeniy Polyakov to Cc ].

v2:
- EXPORT_SYMBOL(kzfree), thanks linker
- remove superfluous NULL checks, thanks Pekka
- mention `security' in the description

Hannes

 arch/s390/crypto/prng.c             |    3 +--
 crypto/api.c                        |    5 +----
 drivers/md/dm-crypt.c               |    6 ++----
 drivers/s390/crypto/zcrypt_pcixcc.c |    3 +--
 drivers/usb/host/hwa-hc.c           |    3 +--
 drivers/usb/wusbcore/cbaf.c         |    3 +--
 fs/cifs/connect.c                   |    6 +-----
 fs/cifs/misc.c                      |   10 ++--------
 fs/ecryptfs/keystore.c              |    3 +--
 fs/ecryptfs/messaging.c             |    3 +--
 include/linux/slab.h                |    1 +
 mm/util.c                           |   20 ++++++++++++++++++++
 12 files changed, 33 insertions(+), 33 deletions(-)

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>