ima: remove redundant initialization of pointer 'file'.
authorColin Ian King <colin.i.king@gmail.com>
Tue, 1 Mar 2022 12:07:32 +0000 (12:07 +0000)
committerMimi Zohar <zohar@linux.ibm.com>
Mon, 4 Apr 2022 20:27:44 +0000 (16:27 -0400)
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>
security/integrity/ima/ima_main.c

index 3d3f8c5..1aebf63 100644 (file)
@@ -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;