diff options
author | Colin Ian King <colin.i.king@gmail.com> | 2022-03-01 13:07:32 +0100 |
---|---|---|
committer | Mimi Zohar <zohar@linux.ibm.com> | 2022-04-04 22:27:44 +0200 |
commit | 29d1c2b47ef3b39a5bc58e060a4ee23ccf3d052e (patch) | |
tree | 2598db948ad77aafa43fdb37ab693076ca4003ae /security | |
parent | Linux 5.18-rc1 (diff) | |
download | linux-29d1c2b47ef3b39a5bc58e060a4ee23ccf3d052e.tar.xz linux-29d1c2b47ef3b39a5bc58e060a4ee23ccf3d052e.zip |
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')
-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; |