Commit 27ba8512 authored by David Howells's avatar David Howells Committed by Trond Myklebust

NFS: Fix error handling

Fix an error handling problem: nfs_put_client() can be given a NULL pointer if
nfs_free_server() is asked to destroy a partially initialised record.
Signed-Off-By: default avatarDavid Howells <dhowells@redhat.com>
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent 6aaca566
...@@ -208,6 +208,9 @@ static void nfs_free_client(struct nfs_client *clp) ...@@ -208,6 +208,9 @@ static void nfs_free_client(struct nfs_client *clp)
*/ */
void nfs_put_client(struct nfs_client *clp) void nfs_put_client(struct nfs_client *clp)
{ {
if (!clp)
return;
dprintk("--> nfs_put_client({%d})\n", atomic_read(&clp->cl_count)); dprintk("--> nfs_put_client({%d})\n", atomic_read(&clp->cl_count));
if (atomic_dec_and_lock(&clp->cl_count, &nfs_client_lock)) { if (atomic_dec_and_lock(&clp->cl_count, &nfs_client_lock)) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment