On Sep 4, 2012, at 9:03 PM, NeilBrown wrote: > While reviewing I noticed: > > if (value > rs->md.raid_disks) { > rs->ti->error = "Invalid rebuild index given"; > return -EINVAL; > } > > (in the 'rebuild' handling). That should be 'value >= rs->md.raid_disks' > shouldn't it? > If you agree, please send a patch. Yes. Thank-you for catching that. Patch coming shortly. brassow -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html