virtio_pci: Prevent double-free of pci regions after device hot-unplug
authorAmit Shah <amit.shah@redhat.com>
Mon, 14 Mar 2011 12:15:02 +0000 (17:15 +0530)
committerRusty Russell <rusty@rustcorp.com.au>
Thu, 21 Apr 2011 13:27:00 +0000 (22:27 +0930)
commit31a3ddda166cda86d2b5111e09ba4bda5239fae6
tree9cd6af0a5a7b59b4452db2fa84545ce782c4006b
parentb3258ff1d6086bd2b9eeb556844a868ad7d49bc8
virtio_pci: Prevent double-free of pci regions after device hot-unplug

In the case where a virtio-console port is in use (opened by a program)
and a virtio-console device is removed, the port is kept around but all
the virtio-related state is assumed to be gone.

When the port is finally released (close() called), we call
device_destroy() on the port's device.  This results in the parent
device's structures to be freed as well.  This includes the PCI regions
for the virtio-console PCI device.

Once this is done, however, virtio_pci_release_dev() kicks in, as the
last ref to the virtio device is now gone, and attempts to do

     pci_iounmap(pci_dev, vp_dev->ioaddr);
     pci_release_regions(pci_dev);
     pci_disable_device(pci_dev);

which results in a double-free warning.

Move the code that releases regions, etc., to the virtio_pci_remove()
function, and all that's now left in release_dev is the final freeing of
the vp_dev.

Signed-off-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
drivers/virtio/virtio_pci.c