diff options
author | Colin Ian King <colin.i.king@gmail.com> | 2022-03-01 12:07:32 +0000 |
---|---|---|
committer | Mimi Zohar <zohar@linux.ibm.com> | 2022-04-04 16:27:44 -0400 |
commit | 29d1c2b47ef3b39a5bc58e060a4ee23ccf3d052e (patch) | |
tree | 2598db948ad77aafa43fdb37ab693076ca4003ae | |
parent | 3123109284176b1532874591f7c81f3837bbdc17 (diff) | |
download | linux-29d1c2b47ef3b39a5bc58e060a4ee23ccf3d052e.tar.bz2 |
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>
-rw-r--r-- | security/integrity/ima/ima_main.c | 2 |
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; |