diff options
author | NeilBrown <neilb@suse.de> | 2009-07-01 12:27:21 +1000 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2009-07-01 12:27:21 +1000 |
commit | 1ec22eb2b4a2e1a763106bce36b11c02eaa84e61 (patch) | |
tree | 2aa6e6539342a28263417f6a2f2551d541e9ead9 /drivers/md | |
parent | b8d966efd9a46a9a35beac50cbff6e30565125ef (diff) | |
download | kernel_samsung_crespo-1ec22eb2b4a2e1a763106bce36b11c02eaa84e61.zip kernel_samsung_crespo-1ec22eb2b4a2e1a763106bce36b11c02eaa84e61.tar.gz kernel_samsung_crespo-1ec22eb2b4a2e1a763106bce36b11c02eaa84e61.tar.bz2 |
md: fix error path when duplicate name is found on md device creation.
When an md device is created by name (rather than number) we need to
check that the name is not already in use. If this check finds a
duplicate, we return an error without dropping the lock or freeing
the newly create mddev.
This patch fixes that.
Cc: stable@kernel.org
Found-by: Jiri Slaby <jirislaby@gmail.com>
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md')
-rw-r--r-- | drivers/md/md.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 2166af8..58bee23 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -3862,6 +3862,8 @@ static int md_alloc(dev_t dev, char *name) if (mddev2->gendisk && strcmp(mddev2->gendisk->disk_name, name) == 0) { spin_unlock(&all_mddevs_lock); + mutex_unlock(&disks_mutex); + mddev_put(mddev); return -EEXIST; } spin_unlock(&all_mddevs_lock); |