]> git.baikalelectronics.ru Git - kernel.git/commit
dm table: reject devices without request fns
authorMilan Broz <mbroz@redhat.com>
Sun, 29 May 2011 12:02:52 +0000 (13:02 +0100)
committerAlasdair G Kergon <agk@redhat.com>
Sun, 29 May 2011 12:02:52 +0000 (13:02 +0100)
commitc2c0bf0755fbd6d4158f95bf2f02512c8962bbf9
tree442641730674c130ec228c626b94606ad4f2e3c7
parentb24af0068dc4ba073cf707b23323e96835a91301
dm table: reject devices without request fns

This patch adds a check that a block device has a request function
defined before it is used.  Otherwise, misconfiguration can cause an oops.

Because we are allowing devices with zero size e.g. an offline multipath
device as in commit ebd7bdcc7e3963a8ee21520264942e7e9ae9eceb
("dm: allow offline devices") there needs to be an additional check
to ensure devices are initialised.  Some block devices, like a loop
device without a backing file, exist but have no request function.

Reproducer is trivial: dm-mirror on unbound loop device
(no backing file on loop devices)

dmsetup create x --table "0 8 mirror core 2 8 sync 2 /dev/loop0 0 /dev/loop1 0"

and mirror resync will immediatelly cause OOps.

BUG: unable to handle kernel NULL pointer dereference at   (null)
 ? generic_make_request+0x2bd/0x590
 ? kmem_cache_alloc+0xad/0x190
 submit_bio+0x53/0xe0
 ? bio_add_page+0x3b/0x50
 dispatch_io+0x1ca/0x210 [dm_mod]
 ? read_callback+0x0/0xd0 [dm_mirror]
 dm_io+0xbb/0x290 [dm_mod]
 do_mirror+0x1e0/0x748 [dm_mirror]

Signed-off-by: Milan Broz <mbroz@redhat.com>
Reported-by: Zdenek Kabelac <zkabelac@redhat.com>
Acked-by: Mike Snitzer <snitzer@redhat.com>
Cc: stable@kernel.org
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
drivers/md/dm-table.c