diff options
| author | Li Zefan <lizf@cn.fujitsu.com> | 2012-01-09 07:36:28 +0100 |
|---|---|---|
| committer | Li Zefan <lizf@cn.fujitsu.com> | 2012-01-11 03:26:36 +0100 |
| commit | 706efc6630c2722602541a6a2fc5900a4e38456a (patch) | |
| tree | bba227c9b49146e6c1bb6b529224088dbb9b8811 /fs/btrfs/free-space-cache.c | |
| parent | Btrfs: avoid possible NULL deref in io_ctl_drop_pages() (diff) | |
| download | linux-706efc6630c2722602541a6a2fc5900a4e38456a.tar.xz linux-706efc6630c2722602541a6a2fc5900a4e38456a.zip | |
Btrfs: check the return value of io_ctl_init()
It can return -ENOMEM.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Diffstat (limited to 'fs/btrfs/free-space-cache.c')
| -rw-r--r-- | fs/btrfs/free-space-cache.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c index 4e55af333e19..e4eb222147cc 100644 --- a/fs/btrfs/free-space-cache.c +++ b/fs/btrfs/free-space-cache.c @@ -637,7 +637,10 @@ int __load_free_space_cache(struct btrfs_root *root, struct inode *inode, if (!num_entries) return 0; - io_ctl_init(&io_ctl, inode, root); + ret = io_ctl_init(&io_ctl, inode, root); + if (ret) + return ret; + ret = readahead_cache(inode); if (ret) goto out; @@ -851,7 +854,9 @@ int __btrfs_write_out_cache(struct btrfs_root *root, struct inode *inode, if (!i_size_read(inode)) return -1; - io_ctl_init(&io_ctl, inode, root); + ret = io_ctl_init(&io_ctl, inode, root); + if (ret) + return -1; /* Get the cluster for this block_group if it exists */ if (block_group && !list_empty(&block_group->cluster_list)) |