Functionally no change, just simplify the ima_detect_file_change() code.

Signed-off-by: Mimi Zohar <[email protected]>
---
@Fred, looking to merge with commit 0824f861605d ("ima: fallback to using
i_version to detect file change")

 security/integrity/ima/ima_main.c | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/security/integrity/ima/ima_main.c 
b/security/integrity/ima/ima_main.c
index 6c49afa8dcc0..5cea53fc36df 100644
--- a/security/integrity/ima/ima_main.c
+++ b/security/integrity/ima/ima_main.c
@@ -194,18 +194,13 @@ static bool ima_detect_file_change(struct ima_iint_cache 
*iint,
        result = vfs_getattr_nosec(&file->f_path, &stat, STATX_CHANGE_COOKIE,
                                   AT_STATX_SYNC_AS_STAT);

-       if (!result && stat.result_mask & STATX_CHANGE_COOKIE &&
-           stat.change_cookie != iint->real_inode.version)
-               return true;
-       else if (!(stat.result_mask & STATX_CHANGE_COOKIE) &&
-                IS_I_VERSION(inode) &&
-                !(inode_eq_iversion(inode, iint->real_inode.version)))
-               return true;
-       else if (!(stat.result_mask & STATX_CHANGE_COOKIE) &&
-                !(IS_I_VERSION(inode)))
-               return true;
-
-       return false;
+       if (!result && stat.result_mask & STATX_CHANGE_COOKIE)
+               return stat.change_cookie != iint->real_inode.version;
+
+       if (IS_I_VERSION(inode))
+               return !inode_eq_iversion(inode, iint->real_inode.version);
+
+       return true;
 }

 static void ima_check_last_writer(struct ima_iint_cache *iint,
--
2.53.0


Reply via email to