summaryrefslogtreecommitdiff
path: root/security/integrity
diff options
context:
space:
mode:
authorColin Ian King <colin.i.king@gmail.com>2022-03-01 12:07:32 +0000
committerMimi Zohar <zohar@linux.ibm.com>2022-04-04 16:27:44 -0400
commit29d1c2b47ef3b39a5bc58e060a4ee23ccf3d052e (patch)
tree2598db948ad77aafa43fdb37ab693076ca4003ae /security/integrity
parent3123109284176b1532874591f7c81f3837bbdc17 (diff)
ima: remove redundant initialization of pointer 'file'.
The pointer 'file' is being initialized with a value that is never read, it is being re-assigned the same value later on closer to where it is being first used. The initialization is redundant and can be removed. Cleans up clang scan build warning: security/integrity/ima/ima_main.c:434:15: warning: Value stored to 'file' during its initialization is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Signed-off-by: Mimi Zohar <zohar@linux.ibm.com>
Diffstat (limited to 'security/integrity')
-rw-r--r--security/integrity/ima/ima_main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/security/integrity/ima/ima_main.c b/security/integrity/ima/ima_main.c
index 3d3f8c5c502b..1aebf63ad7a6 100644
--- a/security/integrity/ima/ima_main.c
+++ b/security/integrity/ima/ima_main.c
@@ -432,7 +432,7 @@ int ima_file_mmap(struct file *file, unsigned long prot)
int ima_file_mprotect(struct vm_area_struct *vma, unsigned long prot)
{
struct ima_template_desc *template = NULL;
- struct file *file = vma->vm_file;
+ struct file *file;
char filename[NAME_MAX];
char *pathbuf = NULL;
const char *pathname = NULL;