Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Li zeming <zeming@nfschina.com>
To:     viro@zeniv.linux.org.uk
Cc:     linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
        Li zeming <zeming@nfschina.com>
Subject: [PATCH] filesystems: Simplify if conditional statements
Date:   Thu,  5 Jan 2023 14:18:31 +0800
Message-Id: <20230105061831.3516-1-zeming@nfschina.com>
Precedence: bulk

When the * p pointer is null, assign a value to res; otherwise, do not
execute the content in the conditional statement block.

Signed-off-by: Li zeming <zeming@nfschina.com>
---
 fs/filesystems.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/fs/filesystems.c b/fs/filesystems.c
index 58b9067b2391..1839dcd6cfbd 100644
--- a/fs/filesystems.c
+++ b/fs/filesystems.c
@@ -71,7 +71,7 @@ static struct file_system_type **find_filesystem(const char *name, unsigned len)
  
 int register_filesystem(struct file_system_type * fs)
 {
-	int res = 0;
+	int res = -EBUSY;
 	struct file_system_type ** p;
 
 	if (fs->parameters &&
@@ -83,10 +83,10 @@ int register_filesystem(struct file_system_type * fs)
 		return -EBUSY;
 	write_lock(&file_systems_lock);
 	p = find_filesystem(fs->name, strlen(fs->name));
-	if (*p)
-		res = -EBUSY;
-	else
+	if (!*p) {
+		res = 0;
 		*p = fs;
+	}
 	write_unlock(&file_systems_lock);
 	return res;
 }
-- 
2.18.2