lkml.org 
[lkml]   [2013]   [Nov]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 2/2] ima: directory integrity protection implementation
Date
This patch provides implmentation of directory integrity protection hooks.

This patch implements ima_dir_check() and ima_dir_update() hooks.

ima_dir_check() verifies the directory integrity during the initial path
lookup, when the dentry is just being created and may block. It allocates
the needed data structures and performs the integrity verification.
The results of which are cached. Subsequent calls mostly happen under
RCU locking, when the code may not block, and returns immediately with
the cached verification status. So ima_dir_check() does not interrupt
RCU path walk.

Directory hash is a hash over the list of directory entries, that includes
name, ino and d_type. ima_dir_check() caclculates a directory hash and compaires
it against good reference value stored in 'security.ima' extended attribute.

ima_dir_update() is called when directory content is changing, and updates
the directory hash.

Signed-off-by: Dmitry Kasatkin <d.kasatkin@samsung.com>
---
security/integrity/ima/Kconfig | 10 +
security/integrity/ima/Makefile | 1 +
security/integrity/ima/ima.h | 3 +-
security/integrity/ima/ima_dir.c | 358 ++++++++++++++++++++++++++++++++++++
security/integrity/ima/ima_policy.c | 2 +
5 files changed, 373 insertions(+), 1 deletion(-)
create mode 100644 security/integrity/ima/ima_dir.c

diff --git a/security/integrity/ima/Kconfig b/security/integrity/ima/Kconfig
index dad8d4c..ec4ef2a 100644
--- a/security/integrity/ima/Kconfig
+++ b/security/integrity/ima/Kconfig
@@ -131,3 +131,13 @@ config IMA_TRUSTED_KEYRING
help
This option requires that all keys added to the _ima
keyring be signed by a key on the system trusted keyring.
+
+config IMA_APPRAISE_DIRECTORIES
+ bool "Performs directory integrity appraisal"
+ depends on IMA_APPRAISE
+ default n
+ help
+ This option enables directory integrity appraisal.
+
+ If unsure, say N.
+
diff --git a/security/integrity/ima/Makefile b/security/integrity/ima/Makefile
index d79263d..518450a 100644
--- a/security/integrity/ima/Makefile
+++ b/security/integrity/ima/Makefile
@@ -8,3 +8,4 @@ obj-$(CONFIG_IMA) += ima.o
ima-y := ima_fs.o ima_queue.o ima_init.o ima_main.o ima_crypto.o ima_api.o \
ima_policy.o ima_template.o ima_template_lib.o
ima-$(CONFIG_IMA_APPRAISE) += ima_appraise.o
+ima-$(CONFIG_IMA_APPRAISE_DIRECTORIES) += ima_dir.o
diff --git a/security/integrity/ima/ima.h b/security/integrity/ima/ima.h
index 6b326a2..fa2dd8b 100644
--- a/security/integrity/ima/ima.h
+++ b/security/integrity/ima/ima.h
@@ -160,7 +160,8 @@ struct integrity_iint_cache *integrity_iint_insert(struct inode *inode);
struct integrity_iint_cache *integrity_iint_find(struct inode *inode);

/* IMA policy related functions */
-enum ima_hooks { FILE_CHECK = 1, MMAP_CHECK, BPRM_CHECK, MODULE_CHECK, POST_SETATTR };
+enum ima_hooks { FILE_CHECK = 1, MMAP_CHECK, BPRM_CHECK, MODULE_CHECK,
+ POST_SETATTR, DIR_CHECK };

int ima_match_policy(struct dentry *dentry, enum ima_hooks func, int mask,
int flags);
diff --git a/security/integrity/ima/ima_dir.c b/security/integrity/ima/ima_dir.c
new file mode 100644
index 0000000..8cc3589
--- /dev/null
+++ b/security/integrity/ima/ima_dir.c
@@ -0,0 +1,358 @@
+/*
+ * Copyright (C) 2011,2012 Intel Corporation
+ * 2013 Samsung Electronics
+ *
+ * Authors:
+ * Dmitry Kasatkin <d.kasatkin@samsung.com>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation, version 2 of the
+ * License.
+ *
+ * File: ima_dir.c
+ * implements the IMA directories hooks: ima_dir_check, ima_dir_update.
+ */
+
+#define pr_fmt(fmt) KBUILD_MODNAME ": %s: " fmt, __func__
+
+#include <linux/module.h>
+#include <linux/file.h>
+#include <linux/mount.h>
+#include <linux/xattr.h>
+#include <linux/ima.h>
+#include <linux/scatterlist.h>
+#include <crypto/hash.h>
+
+#include "ima.h"
+
+static int ima_dir_enabled = 1;
+
+static int __init ima_dir_setup(char *str)
+{
+ if (strncmp(str, "off", 3) == 0)
+ ima_dir_enabled = 0;
+ return 1;
+}
+
+__setup("ima_dir=", ima_dir_setup);
+
+struct readdir_callback {
+ struct dir_context ctx;
+ struct shash_desc *shash;
+};
+
+static int ima_filldir(void *__buf, const char *name, int namelen,
+ loff_t offset, u64 ino, unsigned int d_type)
+{
+ struct readdir_callback *ctx = __buf;
+ struct shash_desc *shash = ctx->shash;
+ int rc;
+
+ rc = crypto_shash_update(shash, name, namelen);
+ rc |= crypto_shash_update(shash, (const u8 *)&ino, sizeof(ino));
+ rc |= crypto_shash_update(shash, (const u8 *)&d_type, sizeof(d_type));
+
+ return rc;
+}
+
+static int ima_calc_dir_hash_tfm(struct path *path, struct file *file,
+ struct ima_digest_data *hash,
+ struct crypto_shash *tfm)
+{
+ struct inode *inode = path->dentry->d_inode;
+ int rc = -ENOTDIR, opened = 0;
+ struct {
+ struct shash_desc shash;
+ char ctx[crypto_shash_descsize(tfm)];
+ } desc;
+ struct readdir_callback buf = {
+ .ctx.actor = ima_filldir,
+ .shash = &desc.shash
+ };
+
+ if (IS_DEADDIR(inode))
+ return -ENOENT;
+
+ if (!file) {
+ file = dentry_open(path, O_RDONLY, current->cred);
+ if (IS_ERR(file))
+ return PTR_ERR(file);
+ opened = 1;
+ }
+
+ if (!file->f_op || !file->f_op->iterate)
+ goto out;
+
+ /* Directory can only be opened for reading? */
+ WARN_ON(!(file->f_mode & FMODE_READ));
+
+ desc.shash.tfm = tfm;
+ desc.shash.flags = 0;
+
+ rc = crypto_shash_init(&desc.shash);
+ if (rc != 0)
+ goto out;
+
+ /* we do not use iterate_dir() because it locks dir i_mutex,
+ which is already locked by our call path */
+ WARN(buf.ctx.pos, "ctx.pos is not NULL");
+ rc = file->f_op->iterate(file, &buf.ctx);
+ if (rc)
+ goto out;
+
+ hash->length = crypto_shash_digestsize(tfm);
+ rc = crypto_shash_final(&desc.shash, hash->digest);
+
+out:
+ if (opened)
+ fput(file);
+ return rc;
+}
+
+int ima_calc_dir_hash(struct path *path, struct file *file,
+ struct ima_digest_data *hash)
+{
+ struct crypto_shash *tfm;
+ int rc;
+
+ tfm = ima_alloc_tfm(hash->algo);
+ if (IS_ERR(tfm))
+ return PTR_ERR(tfm);
+
+ rc = ima_calc_dir_hash_tfm(path, file, hash, tfm);
+
+ ima_free_tfm(tfm);
+
+ return rc;
+}
+
+
+static int ima_dir_collect(struct integrity_iint_cache *iint,
+ struct path *path, struct file *file,
+ struct evm_ima_xattr_data **xattr_value,
+ int *xattr_len)
+{
+ struct dentry *dentry = path->dentry;
+ struct inode *inode = dentry->d_inode;
+ int rc = -EINVAL;
+ struct {
+ struct ima_digest_data hdr;
+ char digest[IMA_MAX_DIGEST_SIZE];
+ } hash;
+
+ if (xattr_value)
+ *xattr_len = ima_read_xattr(dentry, xattr_value);
+
+ if (iint->flags & IMA_COLLECTED)
+ return 0;
+
+ /* use default hash algorithm */
+ hash.hdr.algo = ima_hash_algo;
+
+ if (xattr_value)
+ ima_get_hash_algo(*xattr_value, *xattr_len, &hash.hdr);
+
+ switch (inode->i_mode & S_IFMT) {
+ case S_IFDIR:
+ rc = ima_calc_dir_hash(path, file, &hash.hdr);
+ break;
+ default:
+ pr_debug("UKNOWN: dentry: %s, 0%o\n",
+ dentry->d_name.name, inode->i_mode & S_IFMT);
+ break;
+ }
+
+ if (!rc) {
+ int length = sizeof(hash.hdr) + hash.hdr.length;
+ void *tmpbuf = krealloc(iint->ima_hash, length, GFP_NOFS);
+ if (tmpbuf) {
+ iint->ima_hash = tmpbuf;
+ memcpy(iint->ima_hash, &hash, length);
+ } else
+ rc = -ENOMEM;
+ }
+
+ if (!rc)
+ iint->flags |= IMA_COLLECTED;
+ else
+ integrity_audit_msg(AUDIT_INTEGRITY_DATA, inode,
+ dentry->d_name.name,
+ "collect_data", "failed", rc, 0);
+ return rc;
+}
+
+static int dir_measurement(struct path *path, struct file *file, int mask)
+{
+ struct dentry *dentry = path->dentry;
+ struct inode *inode = dentry->d_inode;
+ struct integrity_iint_cache *iint;
+ char *pathbuf = NULL;
+ const char *pathname;
+ int rc = 0, action, xattr_len = 0, func = DIR_CHECK;
+ struct evm_ima_xattr_data *xattr_value = NULL;
+
+ if (!ima_dir_enabled || !ima_initialized)
+ return 0;
+
+ if (IS_IMA(inode)) {
+ /* inode was already appraised or it is pending... */
+ iint = integrity_iint_find(inode);
+ BUG_ON(!iint);
+
+ action = iint->flags & IMA_DO_MASK;
+ action &= ~((iint->flags & IMA_DONE_MASK) >> 1);
+
+ if (!action)
+ goto out_unlocked;
+
+ if (mask & MAY_NOT_BLOCK)
+ return -ECHILD;
+
+ mutex_lock(&inode->i_mutex);
+ } else {
+ /* Determine if in appraise/measurement policy,
+ * returns IMA_MEASURE, IMA_APPRAISE bitmask. */
+ action = ima_must_appraise(dentry, mask, DIR_CHECK);
+ if (!action)
+ return 0;
+
+ if (mask & MAY_NOT_BLOCK)
+ return -ECHILD;
+ if (action < 0)
+ return action;
+
+ mutex_lock(&inode->i_mutex);
+
+ iint = integrity_inode_get(inode);
+ if (!iint) {
+ rc = -ENOMEM;
+ goto out_locked;
+ }
+
+ iint->flags |= action;
+ action &= IMA_DO_MASK;
+ }
+
+ action &= ~((iint->flags & IMA_DONE_MASK) >> 1);
+
+ /* we only appraise, no other action bits */
+ if (!action)
+ goto out_locked;
+
+ rc = ima_dir_collect(iint, path, file, &xattr_value, &xattr_len);
+ if (rc)
+ goto out_locked;
+
+ pathname = ima_d_path(path, &pathbuf);
+
+ rc = ima_appraise_measurement(func, iint, dentry, pathname,
+ xattr_value, xattr_len);
+ kfree(pathbuf);
+out_locked:
+ mutex_unlock(&inode->i_mutex);
+out_unlocked:
+ if (ima_appraise & IMA_APPRAISE_ENFORCE)
+ return rc ? -EACCES : 0;
+ return 0;
+}
+
+/**
+ * ima_dir_check: verifies directory integrity
+ * @dir: path to verify
+ * @return: error code if appraisal enforced, 0 otherwise
+ *
+ */
+int ima_dir_check(struct path *dir, int mask)
+{
+ BUG_ON(!S_ISDIR(dir->dentry->d_inode->i_mode));
+
+ return dir_measurement(dir, NULL, mask);
+}
+EXPORT_SYMBOL_GPL(ima_dir_check);
+
+int ima_special_check(struct file *file, int mask)
+{
+ if (!S_ISDIR(file->f_dentry->d_inode->i_mode))
+ return 0;
+ return dir_measurement(&file->f_path, file, mask);
+}
+
+static void ima_dir_update_xattr(struct integrity_iint_cache *iint,
+ struct path *path)
+{
+ struct dentry *dentry = path->dentry;
+ struct inode *inode = NULL;
+ int rc;
+
+ if (!iint) {
+ /* if iint is NULL, then we allocated iint for new directory */
+ int action;
+
+ inode = dentry->d_inode;
+
+ /* Determine if in appraise/measurement policy */
+ action = ima_must_appraise(dentry, MAY_READ, DIR_CHECK);
+ if (action <= 0)
+ return;
+
+ mutex_lock(&inode->i_mutex);
+ iint = integrity_inode_get(inode);
+ if (!iint)
+ goto out;
+
+ /* set new inode as measured or/and appraised */
+ action &= IMA_DO_MASK;
+ iint->flags |= action | (action << 1);
+ iint->ima_file_status = INTEGRITY_PASS;
+ }
+
+ rc = ima_dir_collect(iint, path, NULL, NULL, NULL);
+ if (!rc)
+ ima_fix_xattr(dentry, iint);
+out:
+ if (inode)
+ mutex_unlock(&inode->i_mutex);
+}
+
+/**
+ * ima_dir_update - update directory integrity information
+ * @dir: path to update
+ * newdir: dir entry, which added
+ *
+ * It is called when directory content has changed,
+ * and is used to re-calculate and update integrity data.
+ * It is called with dir i_mutex locked.
+ */
+void ima_dir_update(struct path *dir, struct dentry *dentry)
+{
+ struct inode *inode = dir->dentry->d_inode;
+ struct integrity_iint_cache *iint;
+
+ if (!ima_dir_enabled || !ima_initialized)
+ return;
+
+ WARN(IS_PRIVATE(inode), "PRIVATE\n");
+
+ if (unlikely(IS_PRIVATE(inode)))
+ return;
+
+ iint = integrity_iint_find(inode);
+ if (!iint)
+ return;
+
+ if (dentry) {
+ /* new entry -> set initial security.ima value */
+ struct path path = { .mnt = dir->mnt, .dentry = dentry };
+ BUG_ON(!dentry->d_inode);
+ ima_dir_update_xattr(NULL, &path);
+ }
+
+ /* do not reset flags for directories, correct ?
+ iint->flags &= ~(IMA_COLLECTED | IMA_APPRAISED | IMA_MEASURED);
+ */
+ iint->flags &= ~IMA_COLLECTED;
+ if (iint->flags & IMA_APPRAISE)
+ ima_dir_update_xattr(iint, dir);
+}
+EXPORT_SYMBOL_GPL(ima_dir_update);
diff --git a/security/integrity/ima/ima_policy.c b/security/integrity/ima/ima_policy.c
index 317826d..9051e3f 100644
--- a/security/integrity/ima/ima_policy.c
+++ b/security/integrity/ima/ima_policy.c
@@ -516,6 +516,8 @@ static int ima_parse_rule(char *rule, struct ima_rule_entry *entry)
entry->func = MMAP_CHECK;
else if (strcmp(args[0].from, "BPRM_CHECK") == 0)
entry->func = BPRM_CHECK;
+ else if (strcmp(args[0].from, "DIR_CHECK") == 0)
+ entry->func = DIR_CHECK;
else
result = -EINVAL;
if (!result)
--
1.8.3.2


\
 
 \ /
  Last update: 2013-11-18 21:41    [W:0.116 / U:0.312 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site