lsm: Use IS_ERR_OR_NULL() helper function

Hongbo Li · pcmoore · commit ce4a60592ee0 · 2024-08-29T11:12:13.000-04:00
Use the IS_ERR_OR_NULL() helper instead of open-coding a
NULL and an error pointer checks to simplify the code and
improve readability.

Signed-off-by: Hongbo Li &lt;lihongbo22@huawei.com&gt;
Signed-off-by: Paul Moore &lt;paul@paul-moore.com&gt;
diff --git a/security/inode.c b/security/inode.c
@@ -296,7 +296,7 @@ void securityfs_remove(struct dentry *dentry)
 {
 	struct inode *dir;
 
-	if (!dentry || IS_ERR(dentry))
+	if (IS_ERR_OR_NULL(dentry))
 		return;
 
 	dir = d_inode(dentry->d_parent);

Original file line number	Diff line number	Diff line change
`@@ -296,7 +296,7 @@ void securityfs_remove(struct dentry *dentry)`
`296`	`296`	`{`
`297`	`297`	`struct inode *dir;`
`298`	`298`
`299`		`- if (!dentry \|\| IS_ERR(dentry))`
	`299`	`+ if (IS_ERR_OR_NULL(dentry))`
`300`	`300`	`return;`
`301`	`301`
`302`	`302`	`dir = d_inode(dentry->d_parent);`