Commit 7bc5f447 authored by Roberto Sassu's avatar Roberto Sassu Committed by Mimi Zohar

ima: define new function ima_alloc_init_template() to API

Instead of allocating and initializing the template entry from multiple
places (eg. boot aggregate, violation, and regular measurements), this
patch defines a new function called ima_alloc_init_template().  The new
function allocates and initializes the measurement entry with the inode
digest and the filename.

In respect to the current behavior, it truncates the file name passed
in the 'filename' argument if the latter's size is greater than 255 bytes
and the passed file descriptor is NULL.

Changelog:
- initialize 'hash' variable for non TPM case - Mimi
- conform to expectation for 'iint' to be defined as a pointer. - Mimi
- add missing 'file' dependency for recalculating file hash. - Mimi
Signed-off-by: default avatarRoberto Sassu <roberto.sassu@polito.it>
Signed-off-by: default avatarMimi Zohar <zohar@linux.vnet.ibm.com>
parent 9803d413
...@@ -107,6 +107,9 @@ void ima_store_measurement(struct integrity_iint_cache *iint, struct file *file, ...@@ -107,6 +107,9 @@ void ima_store_measurement(struct integrity_iint_cache *iint, struct file *file,
const unsigned char *filename); const unsigned char *filename);
void ima_audit_measurement(struct integrity_iint_cache *iint, void ima_audit_measurement(struct integrity_iint_cache *iint,
const unsigned char *filename); const unsigned char *filename);
int ima_alloc_init_template(struct integrity_iint_cache *iint,
struct file *file, const unsigned char *filename,
struct ima_template_entry **entry);
int ima_store_template(struct ima_template_entry *entry, int violation, int ima_store_template(struct ima_template_entry *entry, int violation,
struct inode *inode, const unsigned char *filename); struct inode *inode, const unsigned char *filename);
void ima_template_show(struct seq_file *m, void *e, enum ima_show_type show); void ima_template_show(struct seq_file *m, void *e, enum ima_show_type show);
......
...@@ -23,6 +23,62 @@ ...@@ -23,6 +23,62 @@
static const char *IMA_TEMPLATE_NAME = "ima"; static const char *IMA_TEMPLATE_NAME = "ima";
/*
* ima_alloc_init_template - create and initialize a new template entry
*/
int ima_alloc_init_template(struct integrity_iint_cache *iint,
struct file *file, const unsigned char *filename,
struct ima_template_entry **entry)
{
struct ima_template_entry *e;
int result = 0;
e = kzalloc(sizeof(**entry), GFP_NOFS);
if (!e)
return -ENOMEM;
memset(&(e)->template, 0, sizeof(e->template));
if (!iint) /* IMA measurement violation entry */
goto out;
if (iint->ima_hash->algo != ima_hash_algo) {
struct inode *inode;
struct {
struct ima_digest_data hdr;
char digest[IMA_MAX_DIGEST_SIZE];
} hash;
if (!file) {
result = -EINVAL;
goto out_free;
}
inode = file_inode(file);
hash.hdr.algo = ima_hash_algo;
hash.hdr.length = SHA1_DIGEST_SIZE;
result = ima_calc_file_hash(file, &hash.hdr);
if (result) {
integrity_audit_msg(AUDIT_INTEGRITY_DATA, inode,
filename, "collect_data",
"failed", result, 0);
goto out_free;
} else
memcpy(e->template.digest, hash.hdr.digest,
hash.hdr.length);
} else
memcpy(e->template.digest, iint->ima_hash->digest,
iint->ima_hash->length);
out:
strcpy(e->template.file_name,
(strlen(filename) > IMA_EVENT_NAME_LEN_MAX && file != NULL) ?
file->f_dentry->d_name.name : filename);
*entry = e;
return 0;
out_free:
kfree(e);
return result;
}
/* /*
* ima_store_template - store ima template measurements * ima_store_template - store ima template measurements
* *
...@@ -90,13 +146,11 @@ void ima_add_violation(struct file *file, const unsigned char *filename, ...@@ -90,13 +146,11 @@ void ima_add_violation(struct file *file, const unsigned char *filename,
/* can overflow, only indicator */ /* can overflow, only indicator */
atomic_long_inc(&ima_htable.violations); atomic_long_inc(&ima_htable.violations);
entry = kmalloc(sizeof(*entry), GFP_KERNEL); result = ima_alloc_init_template(NULL, file, filename, &entry);
if (!entry) { if (result < 0) {
result = -ENOMEM; result = -ENOMEM;
goto err_out; goto err_out;
} }
memset(&entry->template, 0, sizeof(entry->template));
strncpy(entry->template.file_name, filename, IMA_EVENT_NAME_LEN_MAX);
result = ima_store_template(entry, violation, inode, filename); result = ima_store_template(entry, violation, inode, filename);
if (result < 0) if (result < 0)
kfree(entry); kfree(entry);
...@@ -220,34 +274,12 @@ void ima_store_measurement(struct integrity_iint_cache *iint, ...@@ -220,34 +274,12 @@ void ima_store_measurement(struct integrity_iint_cache *iint,
if (iint->flags & IMA_MEASURED) if (iint->flags & IMA_MEASURED)
return; return;
entry = kmalloc(sizeof(*entry), GFP_KERNEL); result = ima_alloc_init_template(iint, file, filename, &entry);
if (!entry) { if (result < 0) {
integrity_audit_msg(AUDIT_INTEGRITY_PCR, inode, filename, integrity_audit_msg(AUDIT_INTEGRITY_PCR, inode, filename,
op, audit_cause, result, 0); op, audit_cause, result, 0);
return; return;
} }
memset(&entry->template, 0, sizeof(entry->template));
if (iint->ima_hash->algo != ima_hash_algo) {
struct {
struct ima_digest_data hdr;
char digest[IMA_MAX_DIGEST_SIZE];
} hash;
hash.hdr.algo = ima_hash_algo;
result = ima_calc_file_hash(file, &hash.hdr);
if (result)
integrity_audit_msg(AUDIT_INTEGRITY_DATA, inode,
filename, "collect_data", "failed",
result, 0);
else
memcpy(entry->template.digest, hash.hdr.digest,
hash.hdr.length);
} else
memcpy(entry->template.digest, iint->ima_hash->digest,
iint->ima_hash->length);
strcpy(entry->template.file_name,
(strlen(filename) > IMA_EVENT_NAME_LEN_MAX) ?
file->f_dentry->d_name.name : filename);
result = ima_store_template(entry, violation, inode, filename); result = ima_store_template(entry, violation, inode, filename);
if (!result || result == -EEXIST) if (!result || result == -EEXIST)
......
...@@ -43,34 +43,36 @@ int ima_used_chip; ...@@ -43,34 +43,36 @@ int ima_used_chip;
static void __init ima_add_boot_aggregate(void) static void __init ima_add_boot_aggregate(void)
{ {
struct ima_template_entry *entry; struct ima_template_entry *entry;
struct integrity_iint_cache tmp_iint, *iint = &tmp_iint;
const char *op = "add_boot_aggregate"; const char *op = "add_boot_aggregate";
const char *audit_cause = "ENOMEM"; const char *audit_cause = "ENOMEM";
int result = -ENOMEM; int result = -ENOMEM;
int violation = 1; int violation = 0;
struct { struct {
struct ima_digest_data hdr; struct ima_digest_data hdr;
char digest[TPM_DIGEST_SIZE]; char digest[TPM_DIGEST_SIZE];
} hash; } hash;
entry = kmalloc(sizeof(*entry), GFP_KERNEL); memset(iint, 0, sizeof(*iint));
if (!entry) memset(&hash, 0, sizeof(hash));
goto err_out; iint->ima_hash = &hash.hdr;
iint->ima_hash->algo = HASH_ALGO_SHA1;
iint->ima_hash->length = SHA1_DIGEST_SIZE;
memset(&entry->template, 0, sizeof(entry->template));
strncpy(entry->template.file_name, boot_aggregate_name,
IMA_EVENT_NAME_LEN_MAX);
if (ima_used_chip) { if (ima_used_chip) {
violation = 0;
hash.hdr.algo = HASH_ALGO_SHA1;
result = ima_calc_boot_aggregate(&hash.hdr); result = ima_calc_boot_aggregate(&hash.hdr);
if (result < 0) { if (result < 0) {
audit_cause = "hashing_error"; audit_cause = "hashing_error";
kfree(entry); kfree(entry);
goto err_out; goto err_out;
} }
memcpy(entry->template.digest, hash.hdr.digest,
hash.hdr.length);
} }
result = ima_alloc_init_template(iint, NULL, boot_aggregate_name,
&entry);
if (result < 0)
return;
result = ima_store_template(entry, violation, NULL, result = ima_store_template(entry, violation, NULL,
boot_aggregate_name); boot_aggregate_name);
if (result < 0) if (result < 0)
......
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