diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-13 20:59:33 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-13 20:59:33 -0400 |
commit | fdb1335a82ef1ef9442ac9377796e4e7a69d1ae4 (patch) | |
tree | e661a91753398208d338a089f45e00530475f721 | |
parent | d55e5bd0201a2af0182687882a92c5f95dbccc12 (diff) | |
parent | 2d4f4f3384d4ef4f7c571448e803a1ce721113d5 (diff) |
Merge tag 'md-3.5-fixes' of git://neil.brown.name/md
Pull use-after-free RAID1 bugfix from NeilBrown.
* tag 'md-3.5-fixes' of git://neil.brown.name/md:
md/raid1: fix use-after-free bug in RAID1 data-check code.
-rw-r--r-- | drivers/md/raid1.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c index 8c2754f835ef..240ff3125040 100644 --- a/drivers/md/raid1.c +++ b/drivers/md/raid1.c | |||
@@ -2485,9 +2485,10 @@ static sector_t sync_request(struct mddev *mddev, sector_t sector_nr, int *skipp | |||
2485 | */ | 2485 | */ |
2486 | if (test_bit(MD_RECOVERY_REQUESTED, &mddev->recovery)) { | 2486 | if (test_bit(MD_RECOVERY_REQUESTED, &mddev->recovery)) { |
2487 | atomic_set(&r1_bio->remaining, read_targets); | 2487 | atomic_set(&r1_bio->remaining, read_targets); |
2488 | for (i = 0; i < conf->raid_disks * 2; i++) { | 2488 | for (i = 0; i < conf->raid_disks * 2 && read_targets; i++) { |
2489 | bio = r1_bio->bios[i]; | 2489 | bio = r1_bio->bios[i]; |
2490 | if (bio->bi_end_io == end_sync_read) { | 2490 | if (bio->bi_end_io == end_sync_read) { |
2491 | read_targets--; | ||
2491 | md_sync_acct(bio->bi_bdev, nr_sectors); | 2492 | md_sync_acct(bio->bi_bdev, nr_sectors); |
2492 | generic_make_request(bio); | 2493 | generic_make_request(bio); |
2493 | } | 2494 | } |