diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-02-09 20:18:00 +0300 |
|---|---|---|
| committer | James Morris <james.l.morris@oracle.com> | 2017-02-10 04:43:51 +0300 |
| commit | 52176603795c2ab7e9faf6bb94820da1b726aabd (patch) | |
| tree | b1b7e7c2938d1b9ae0446ed5ce2da554e326d36c /security | |
| parent | 57cb17e764ba0aaa169d07796acce54ccfbc6cae (diff) | |
| download | linux-52176603795c2ab7e9faf6bb94820da1b726aabd.tar.xz | |
KEYS: Use memzero_explicit() for secret data
I don't think GCC has figured out how to optimize the memset() away, but
they might eventually so let's future proof this code a bit.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Acked-by: Mimi Zohar <zohar@linux.vnet.ibm.com>
Signed-off-by: James Morris <james.l.morris@oracle.com>
Diffstat (limited to 'security')
| -rw-r--r-- | security/keys/encrypted-keys/encrypted.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/keys/encrypted-keys/encrypted.c b/security/keys/encrypted-keys/encrypted.c index d7a4969b2dd3..4fb315cddf5b 100644 --- a/security/keys/encrypted-keys/encrypted.c +++ b/security/keys/encrypted-keys/encrypted.c @@ -985,7 +985,7 @@ static void encrypted_destroy(struct key *key) if (!epayload) return; - memset(epayload->decrypted_data, 0, epayload->decrypted_datalen); + memzero_explicit(epayload->decrypted_data, epayload->decrypted_datalen); kfree(key->payload.data[0]); } |