Commit 5c2a640a authored by Stefan Berger's avatar Stefan Berger Committed by Jarkko Sakkinen

ima: Use tpm_default_chip() and call TPM functions with a tpm_chip

Rather than accessing the TPM functions by passing a NULL pointer for
the tpm_chip, which causes a lookup for a suitable chip every time, get a
hold of a tpm_chip and access the TPM functions using it.
Signed-off-by: default avatarStefan Berger <stefanb@linux.vnet.ibm.com>
Signed-off-by: default avatarMimi Zohar <zohar@linux.vnet.ibm.com>
Reviewed-by: default avatarJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Signed-off-by: default avatarJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
parent 58bac8cc
...@@ -56,6 +56,7 @@ extern int ima_policy_flag; ...@@ -56,6 +56,7 @@ extern int ima_policy_flag;
extern int ima_used_chip; extern int ima_used_chip;
extern int ima_hash_algo; extern int ima_hash_algo;
extern int ima_appraise; extern int ima_appraise;
extern struct tpm_chip *ima_tpm_chip;
/* IMA event related data */ /* IMA event related data */
struct ima_event_data { struct ima_event_data {
......
...@@ -634,7 +634,7 @@ static void __init ima_pcrread(int idx, u8 *pcr) ...@@ -634,7 +634,7 @@ static void __init ima_pcrread(int idx, u8 *pcr)
if (!ima_used_chip) if (!ima_used_chip)
return; return;
if (tpm_pcr_read(NULL, idx, pcr) != 0) if (tpm_pcr_read(ima_tpm_chip, idx, pcr) != 0)
pr_err("Error Communicating to TPM chip\n"); pr_err("Error Communicating to TPM chip\n");
} }
......
...@@ -27,6 +27,7 @@ ...@@ -27,6 +27,7 @@
/* name for boot aggregate entry */ /* name for boot aggregate entry */
static const char *boot_aggregate_name = "boot_aggregate"; static const char *boot_aggregate_name = "boot_aggregate";
int ima_used_chip; int ima_used_chip;
struct tpm_chip *ima_tpm_chip;
/* Add the boot aggregate to the IMA measurement list and extend /* Add the boot aggregate to the IMA measurement list and extend
* the PCR register. * the PCR register.
...@@ -106,17 +107,13 @@ void __init ima_load_x509(void) ...@@ -106,17 +107,13 @@ void __init ima_load_x509(void)
int __init ima_init(void) int __init ima_init(void)
{ {
u8 pcr_i[TPM_DIGEST_SIZE];
int rc; int rc;
ima_used_chip = 0; ima_tpm_chip = tpm_default_chip();
rc = tpm_pcr_read(NULL, 0, pcr_i);
if (rc == 0)
ima_used_chip = 1;
ima_used_chip = ima_tpm_chip != NULL;
if (!ima_used_chip) if (!ima_used_chip)
pr_info("No TPM chip found, activating TPM-bypass! (rc=%d)\n", pr_info("No TPM chip found, activating TPM-bypass!\n");
rc);
rc = integrity_init_keyring(INTEGRITY_KEYRING_IMA); rc = integrity_init_keyring(INTEGRITY_KEYRING_IMA);
if (rc) if (rc)
......
...@@ -145,7 +145,7 @@ static int ima_pcr_extend(const u8 *hash, int pcr) ...@@ -145,7 +145,7 @@ static int ima_pcr_extend(const u8 *hash, int pcr)
if (!ima_used_chip) if (!ima_used_chip)
return result; return result;
result = tpm_pcr_extend(NULL, pcr, hash); result = tpm_pcr_extend(ima_tpm_chip, pcr, hash);
if (result != 0) if (result != 0)
pr_err("Error Communicating to TPM chip, result: %d\n", result); pr_err("Error Communicating to TPM chip, result: %d\n", result);
return result; return result;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment