diff options
| author | NeilBrown <neilb@suse.de> | 2010-05-17 11:27:00 +1000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-07-05 11:22:23 -0700 |
| commit | 784400c5bf5ded58e79d935e9d2ddc989ba4fb30 (patch) | |
| tree | 2ebf3118d681e54e1665613f40f0fb39c8affee8 /drivers | |
| parent | cc620515a522a1914962ba11ce99b6157f918b21 (diff) | |
md/linear: avoid possible oops and array stop
commit ef2f80ff7325b2c1888ff02ead28957b5840bf51 upstream.
Since commit ef286f6fa673cd7fb367e1b145069d8dbfcc6081
it has been important that each personality clears
->private in the ->stop() function, or sets it to a
attribute group to be removed.
linear.c doesn't. This can sometimes lead to an oops,
though it doesn't always.
Suitable for 2.6.33-stable and 2.6.34.
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/md/linear.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/md/linear.c b/drivers/md/linear.c index 09437e958235..0a1042b1f83a 100644 --- a/drivers/md/linear.c +++ b/drivers/md/linear.c @@ -282,6 +282,7 @@ static int linear_stop (mddev_t *mddev) rcu_barrier(); blk_sync_queue(mddev->queue); /* the unplug fn references 'conf'*/ kfree(conf); + mddev->private = NULL; return 0; } |