diff options
| author |   Tsutomu Itoh <t-itoh@jp.fujitsu.com> | 2011-01-05 02:32:22 +0000 |
|---|---|---|
| committer |   Chris Mason <chris.mason@oracle.com> | 2011-01-16 11:30:20 -0500 |
| commit | 91ca338d776e0cefb255bf2979b6448febd880f5 (patch) | |
| tree | 5d1d344d6ed565faf5612b5c695f00c04e6f01d8 /fs/btrfs/disk-io.c | |
| parent | btrfs: Don't pass NULL ptr to func that may deref it. (diff) | |
| download | linux-dev-91ca338d776e0cefb255bf2979b6448febd880f5.tar.xz linux-dev-91ca338d776e0cefb255bf2979b6448febd880f5.zip | |
btrfs: check NULL or not
Should check if functions returns NULL or not.
Signed-off-by: Tsutomu Itoh <t-itoh@jp.fujitsu.com>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/disk-io.c')
| -rw-r--r-- | fs/btrfs/disk-io.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index f9efb68fc2e3..a0c37b2ee9ed 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c @@ -353,6 +353,10 @@ static int csum_dirty_buffer(struct btrfs_root *root, struct page *page) WARN_ON(len == 0); eb = alloc_extent_buffer(tree, start, len, page, GFP_NOFS); + if (eb == NULL) { + WARN_ON(1); + goto out; + } ret = btree_read_extent_buffer_pages(root, eb, start + PAGE_CACHE_SIZE, btrfs_header_generation(eb)); BUG_ON(ret); @@ -427,6 +431,10 @@ static int btree_readpage_end_io_hook(struct page *page, u64 start, u64 end, WARN_ON(len == 0); eb = alloc_extent_buffer(tree, start, len, page, GFP_NOFS); + if (eb == NULL) { + ret = -EIO; + goto out; + } found_start = btrfs_header_bytenr(eb); if (found_start != start) { |