diff options
| author | NeilBrown <neilb@suse.de> | 2013-02-21 15:36:38 +1100 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2013-03-20 15:03:18 +0000 |
| commit | ab94b06b25d7678af3dcbb20b46349822607e0ad (patch) | |
| tree | 8836ed9bd97cb70d62160f468e5a747e7453e75d /drivers/md | |
| parent | 84453f1a9d27a3a9be88269cc14ee9099856f268 (diff) | |
md: raid0: fix error return from create_stripe_zones.
commit 58ebb34c49fcfcaa029e4b1c1453d92583900f9a upstream.
Create_stripe_zones returns an error slightly differently to
raid0_run and to raid0_takeover_*.
The error returned used by the second was wrong and an error would
result in mddev->private being set to NULL and sooner or later a
crash.
So never return NULL, return ERR_PTR(err), not NULL from
create_stripe_zones.
This bug has been present since 2.6.35 so the fix is suitable
for any kernel since then.
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/raid0.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c index 0c7bff97fdc1..d3e6f354f878 100644 --- a/drivers/md/raid0.c +++ b/drivers/md/raid0.c @@ -286,7 +286,7 @@ abort: kfree(conf->strip_zone); kfree(conf->devlist); kfree(conf); - *private_conf = NULL; + *private_conf = ERR_PTR(err); return err; } |