]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
virtio: Don't access index after unregister.
authorCornelia Huck <cornelia.huck@de.ibm.com>
Fri, 9 Nov 2012 04:24:12 +0000 (14:54 +1030)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 26 Nov 2012 20:14:06 +0000 (12:14 -0800)
commit01ae49199e6a67075b53150ecd8dfc627a67183b
treeb0254e25c2906f62aa7294e75688a6ccc7d662be
parent17773c7701fe1eb68b8841f215b506dca07a3b6a
virtio: Don't access index after unregister.

commit 237242bddc99041e15a4ca51b8439657cadaff17 upstream.

Virtio wants to release used indices after the corresponding
virtio device has been unregistered. However, virtio does not
hold an extra reference, giving up its last reference with
device_unregister(), making accessing dev->index afterwards
invalid.

I actually saw problems when testing my (not-yet-merged)
virtio-ccw code:

- device_add virtio-net,id=xxx
-> creates device virtio<n> with n>0

- device_del xxx
-> deletes virtio<n>, but calls ida_simple_remove with an
   index of 0

- device_add virtio-net,id=xxx
-> tries to add virtio0, which is still in use...

So let's save the index we want to release before calling
device_unregister().

Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Acked-by: Sjur Brændeland <sjur.brandeland@stericsson.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/virtio/virtio.c