mirror of
https://github.com/torvalds/linux.git
synced 2024-11-12 07:01:57 +00:00
[PATCH] dm: fix find_device race
There is a race between dev_create() and find_device(). If the mdptr has not yet been stored against a device, find_device() needs to behave as though no device was found. It already returns NULL, but there is a dm_put() missing: it must drop the reference dm_get_md() took. The bug was introduced by dm-fix-mapped-device-ref-counting.patch. It manifests itself if another dm ioctl attempts to reference a newly-created device while the device creation ioctl is still running. The consequence is that the device cannot be removed until the machine is rebooted. Certain udev configurations can lead to this happening. Signed-off-by: Alasdair G Kergon <agk@redhat.com> Cc: <dm-devel@redhat.com> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
c06cb8b1c4
commit
bfc5ecdf48
@ -606,9 +606,14 @@ static struct hash_cell *__find_device_hash_cell(struct dm_ioctl *param)
|
||||
return __get_name_cell(param->name);
|
||||
|
||||
md = dm_get_md(huge_decode_dev(param->dev));
|
||||
if (md)
|
||||
mdptr = dm_get_mdptr(md);
|
||||
if (!md)
|
||||
goto out;
|
||||
|
||||
mdptr = dm_get_mdptr(md);
|
||||
if (!mdptr)
|
||||
dm_put(md);
|
||||
|
||||
out:
|
||||
return mdptr;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user