NFSv4: Fix incorrect semaphore release in _nfs4_do_open()
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Fri, 15 Sep 2006 12:11:51 +0000 (08:11 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 19 Sep 2006 15:54:53 +0000 (11:54 -0400)
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/nfs4proc.c

index 153898e1331f714455ee2dc513cc04c5d65bcf1c..b14145b7b87f31a633de6965b031e594e8dd3d60 100644 (file)
@@ -970,7 +970,7 @@ static int _nfs4_do_open(struct inode *dir, struct dentry *dentry, int flags, st
        status = -ENOMEM;
        opendata = nfs4_opendata_alloc(dentry, sp, flags, sattr);
        if (opendata == NULL)
-               goto err_put_state_owner;
+               goto err_release_rwsem;
 
        status = _nfs4_proc_open(opendata);
        if (status != 0)
@@ -989,11 +989,11 @@ static int _nfs4_do_open(struct inode *dir, struct dentry *dentry, int flags, st
        return 0;
 err_opendata_free:
        nfs4_opendata_free(opendata);
+err_release_rwsem:
+       up_read(&clp->cl_sem);
 err_put_state_owner:
        nfs4_put_state_owner(sp);
 out_err:
-       /* Note: clp->cl_sem must be released before nfs4_put_open_state()! */
-       up_read(&clp->cl_sem);
        *res = NULL;
        return status;
 }