diff options
author | Boshi Wang <wangboshi@huawei.com> | 2017-10-20 04:01:03 -0400 |
---|---|---|
committer | Mimi Zohar <zohar@linux.vnet.ibm.com> | 2017-11-08 15:16:36 -0500 |
commit | ebe7c0a7be92bbd34c6ff5b55810546a0ee05bee (patch) | |
tree | 62bd4ad47d06f6c910befaa14465eda0114743e8 /security | |
parent | 0485d066d82c308e28e76b7fc6cdec46ae46eeb6 (diff) |
ima: fix hash algorithm initialization
The hash_setup function always sets the hash_setup_done flag, even
when the hash algorithm is invalid. This prevents the default hash
algorithm defined as CONFIG_IMA_DEFAULT_HASH from being used.
This patch sets hash_setup_done flag only for valid hash algorithms.
Fixes: e7a2ad7eb6f4 "ima: enable support for larger default filedata hash
algorithms"
Signed-off-by: Boshi Wang <wangboshi@huawei.com>
Signed-off-by: Mimi Zohar <zohar@linux.vnet.ibm.com>
Diffstat (limited to 'security')
-rw-r--r-- | security/integrity/ima/ima_main.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/security/integrity/ima/ima_main.c b/security/integrity/ima/ima_main.c index d47f92e97f80..d6ddaad91e82 100644 --- a/security/integrity/ima/ima_main.c +++ b/security/integrity/ima/ima_main.c | |||
@@ -51,6 +51,8 @@ static int __init hash_setup(char *str) | |||
51 | ima_hash_algo = HASH_ALGO_SHA1; | 51 | ima_hash_algo = HASH_ALGO_SHA1; |
52 | else if (strncmp(str, "md5", 3) == 0) | 52 | else if (strncmp(str, "md5", 3) == 0) |
53 | ima_hash_algo = HASH_ALGO_MD5; | 53 | ima_hash_algo = HASH_ALGO_MD5; |
54 | else | ||
55 | return 1; | ||
54 | goto out; | 56 | goto out; |
55 | } | 57 | } |
56 | 58 | ||
@@ -60,6 +62,8 @@ static int __init hash_setup(char *str) | |||
60 | break; | 62 | break; |
61 | } | 63 | } |
62 | } | 64 | } |
65 | if (i == HASH_ALGO__LAST) | ||
66 | return 1; | ||
63 | out: | 67 | out: |
64 | hash_setup_done = 1; | 68 | hash_setup_done = 1; |
65 | return 1; | 69 | return 1; |