]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
NFS: cleanup error handling in nfs4_server_common_setup
authorChristoph Hellwig <hch@lst.de>
Fri, 18 Jul 2025 08:14:46 +0000 (10:14 +0200)
committerTrond Myklebust <trond.myklebust@hammerspace.com>
Tue, 22 Jul 2025 12:10:41 +0000 (08:10 -0400)
Return error directly instead of using a goto label for it.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20250718081509.2607553-2-hch@lst.de
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
fs/nfs/nfs4client.c

index 2e623da1a78794cc07f2885c5d2441b9e80fc083..5943a192f36be7cc00eea553e007582fd9da671e 100644 (file)
@@ -1103,14 +1103,14 @@ static int nfs4_server_common_setup(struct nfs_server *server,
        /* We must ensure the session is initialised first */
        error = nfs4_init_session(server->nfs_client);
        if (error < 0)
-               goto out;
+               return error;
 
        nfs4_server_set_init_caps(server);
 
        /* Probe the root fh to retrieve its FSID and filehandle */
        error = nfs4_get_rootfh(server, mntfh, auth_probe);
        if (error < 0)
-               goto out;
+               return error;
 
        dprintk("Server FSID: %llx:%llx\n",
                        (unsigned long long) server->fsid.major,
@@ -1119,7 +1119,7 @@ static int nfs4_server_common_setup(struct nfs_server *server,
 
        error = nfs_probe_server(server, mntfh);
        if (error < 0)
-               goto out;
+               return error;
 
        nfs4_session_limit_rwsize(server);
        nfs4_session_limit_xasize(server);
@@ -1130,8 +1130,7 @@ static int nfs4_server_common_setup(struct nfs_server *server,
        nfs_server_insert_lists(server);
        server->mount_time = jiffies;
        server->destroy = nfs4_destroy_server;
-out:
-       return error;
+       return 0;
 }
 
 /*