]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
ima: fix memory leak in ima_release_policy
authorEric Richter <erichte@linux.vnet.ibm.com>
Thu, 13 Oct 2016 22:47:36 +0000 (17:47 -0500)
committerMimi Zohar <zohar@linux.vnet.ibm.com>
Mon, 14 Nov 2016 03:50:08 +0000 (22:50 -0500)
When the "policy" securityfs file is opened for read, it is opened as a
sequential file. However, when it is eventually released, there is no
cleanup for the sequential file, therefore some memory is leaked.

This patch adds a call to seq_release() in ima_release_policy() to clean up
the memory when the file is opened for read.

Fixes: 80eae209d63a IMA: allow reading back the current policy
Reported-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Eric Richter <erichte@linux.vnet.ibm.com>
Tested-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Mimi Zohar <zohar@linux.vnet.ibm.com>
security/integrity/ima/ima_fs.c

index c07a3844ea0a290efcee652b3212483e4e58f067..3df46906492dec72a6381aa2a6aa67500e6d7883 100644 (file)
@@ -401,7 +401,7 @@ static int ima_release_policy(struct inode *inode, struct file *file)
        const char *cause = valid_policy ? "completed" : "failed";
 
        if ((file->f_flags & O_ACCMODE) == O_RDONLY)
-               return 0;
+               return seq_release(inode, file);
 
        if (valid_policy && ima_check_policy() < 0) {
                cause = "failed";