summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorYanchuan Nian <ycnian@gmail.com>2012-10-31 16:05:48 +0800
committerTrond Myklebust <Trond.Myklebust@netapp.com>2012-10-31 16:26:25 -0400
commit7175fe90153e6375082d65884fbb41ab3bbb4901 (patch)
tree487e9206cc31f1a214ee43771325193abb4a2b89 /fs
parent8d96b10639fb402357b75b055b1e82a65ff95050 (diff)
downloadop-kernel-dev-7175fe90153e6375082d65884fbb41ab3bbb4901.zip
op-kernel-dev-7175fe90153e6375082d65884fbb41ab3bbb4901.tar.gz
nfs: Check whether a layout pointer is NULL before free it
The new layout pointer in pnfs_find_alloc_layout() may be NULL because of out of memory. we must do some check work, otherwise pnfs_free_layout_hdr() will go wrong because it can not deal with a NULL pointer. Signed-off-by: Yanchuan Nian <ycnian@gmail.com> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/nfs/pnfs.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c
index fe624c9..2878f97 100644
--- a/fs/nfs/pnfs.c
+++ b/fs/nfs/pnfs.c
@@ -925,8 +925,8 @@ pnfs_find_alloc_layout(struct inode *ino,
if (likely(nfsi->layout == NULL)) { /* Won the race? */
nfsi->layout = new;
return new;
- }
- pnfs_free_layout_hdr(new);
+ } else if (new != NULL)
+ pnfs_free_layout_hdr(new);
out_existing:
pnfs_get_layout_hdr(nfsi->layout);
return nfsi->layout;
OpenPOWER on IntegriCloud