NFSv4.1: Fix a reference leak in pnfs_update_layout
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Fri, 21 Sep 2012 01:25:19 +0000 (21:25 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Fri, 28 Sep 2012 20:03:10 +0000 (16:03 -0400)
If we exit after the call to pnfs_find_alloc_layout(), we have to ensure
that we put the struct pnfs_layout_hdr.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/pnfs.c

index 2c59da5511db35448490666b119e5f089a71a3aa..d7a8f03e72959bcab9ed25037ab7216e5c0c2542 100644 (file)
@@ -1067,8 +1067,10 @@ pnfs_update_layout(struct inode *ino,
 
        spin_lock(&ino->i_lock);
        lo = pnfs_find_alloc_layout(ino, ctx, gfp_flags);
-       if (lo == NULL)
-               goto out_unlock;
+       if (lo == NULL) {
+               spin_unlock(&ino->i_lock);
+               goto out;
+       }
 
        /* Do we even need to bother with this? */
        if (test_bit(NFS_LAYOUT_BULK_RECALL, &lo->plh_flags)) {
@@ -1122,6 +1124,7 @@ pnfs_update_layout(struct inode *ino,
                spin_unlock(&clp->cl_lock);
        }
        atomic_dec(&lo->plh_outstanding);
+out_put_layout_hdr:
        pnfs_put_layout_hdr(lo);
 out:
        dprintk("%s: inode %s/%llu pNFS layout segment %s for "
@@ -1135,7 +1138,7 @@ out:
        return lseg;
 out_unlock:
        spin_unlock(&ino->i_lock);
-       goto out;
+       goto out_put_layout_hdr;
 }
 EXPORT_SYMBOL_GPL(pnfs_update_layout);