diff options
author | Tsutomu Itoh <t-itoh@jp.fujitsu.com> | 2012-02-16 01:11:40 (GMT) |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2012-02-16 16:23:16 (GMT) |
commit | 013bd4c336ad0d30e9e41f9cff0dbc1858934e75 (patch) | |
tree | f605d1160c95e3e82016ca8a81e6e1c3af2b7cba | |
parent | 12fc9d0923ca70ae8960bccebac09d5c12f8c4d4 (diff) | |
download | linux-fsl-qoriq-013bd4c336ad0d30e9e41f9cff0dbc1858934e75.tar.xz |
Btrfs: fix return value check of extent_io_ops
This patch adds the check on the return value of extent_io_ops.
Signed-off-by: Tsutomu Itoh <t-itoh@jp.fujitsu.com>
-rw-r--r-- | fs/btrfs/extent_io.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index 89ba79f..b05d35a 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c @@ -2154,9 +2154,10 @@ static int bio_readpage_error(struct bio *failed_bio, struct page *page, "this_mirror=%d, num_copies=%d, in_validation=%d\n", read_mode, failrec->this_mirror, num_copies, failrec->in_validation); - tree->ops->submit_bio_hook(inode, read_mode, bio, failrec->this_mirror, - failrec->bio_flags, 0); - return 0; + ret = tree->ops->submit_bio_hook(inode, read_mode, bio, + failrec->this_mirror, + failrec->bio_flags, 0); + return ret; } /* lots and lots of room for performance fixes in the end_bio funcs */ @@ -2790,9 +2791,12 @@ static int __extent_writepage(struct page *page, struct writeback_control *wbc, delalloc_start = delalloc_end + 1; continue; } - tree->ops->fill_delalloc(inode, page, delalloc_start, - delalloc_end, &page_started, - &nr_written); + ret = tree->ops->fill_delalloc(inode, page, + delalloc_start, + delalloc_end, + &page_started, + &nr_written); + BUG_ON(ret); /* * delalloc_end is already one less than the total * length, so we don't subtract one from |