diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2016-02-12 12:24:48 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-02-12 12:24:48 -0500 |
commit | df8c2723a1eb568d42ebca806bec17377ae7e84c (patch) | |
tree | 03701b18ce9b8b78e60cb91707829df1548c4cf1 | |
parent | 27c9d772e50731dfd682b4ea9459eccec2071c49 (diff) | |
parent | 613317bd212c585c20796c10afe5daaa95d4b0a1 (diff) |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security
Pull crypto fix from James Morris.
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security:
EVM: Use crypto_memneq() for digest comparisons
-rw-r--r-- | security/integrity/evm/evm_main.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/security/integrity/evm/evm_main.c b/security/integrity/evm/evm_main.c index f7160253f17f..e6ea9d4b1de9 100644 --- a/security/integrity/evm/evm_main.c +++ b/security/integrity/evm/evm_main.c | |||
@@ -23,6 +23,7 @@ | |||
23 | #include <linux/integrity.h> | 23 | #include <linux/integrity.h> |
24 | #include <linux/evm.h> | 24 | #include <linux/evm.h> |
25 | #include <crypto/hash.h> | 25 | #include <crypto/hash.h> |
26 | #include <crypto/algapi.h> | ||
26 | #include "evm.h" | 27 | #include "evm.h" |
27 | 28 | ||
28 | int evm_initialized; | 29 | int evm_initialized; |
@@ -148,7 +149,7 @@ static enum integrity_status evm_verify_hmac(struct dentry *dentry, | |||
148 | xattr_value_len, calc.digest); | 149 | xattr_value_len, calc.digest); |
149 | if (rc) | 150 | if (rc) |
150 | break; | 151 | break; |
151 | rc = memcmp(xattr_data->digest, calc.digest, | 152 | rc = crypto_memneq(xattr_data->digest, calc.digest, |
152 | sizeof(calc.digest)); | 153 | sizeof(calc.digest)); |
153 | if (rc) | 154 | if (rc) |
154 | rc = -EINVAL; | 155 | rc = -EINVAL; |