kzfree() v2
Get series with b4
b4 am [email protected]Versions newest first
- 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 files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/7 | slab: introduce kzfree()include/ mm/ | +21 | 16 replies | |
| 2/7 | crypto: use kzfree()crypto/ | +1 | 1 reply | |
| 3/7 | s390: use kzfree()arch/ drivers/ | +2 | 0 replies | |
| 4/7 | md: use kzfree()drivers/ | +2 | 0 replies | |
| 5/7 | usb: use kzfree()drivers/ drivers/ | +2 | 1 reply | |
| 6/7 | cifs: use kzfree()fs/ fs/ | +3 | 0 replies | |
| 7/7 | ecryptfs: use kzfree()fs/ fs/ | +2 | 0 replies |
Reviews trailers per patch
All 7 patches have a Reviewed-by or Acked-by.
| Patch | Pekka Enberg | Tyler Hicks | Steve French | Christoph Lameter | David Vrabel | Herbert Xu |
|---|---|---|---|---|---|---|
| 1/7 | R-b | A-b | ||||
| 2/7 | R-b | A-b | ||||
| 3/7 | R-b | |||||
| 4/7 | R-b | |||||
| 5/7 | R-b | A-b | ||||
| 6/7 | R-b | A-b | ||||
| 7/7 | R-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.
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>