diff options
| author |   Benoît Canet <benoit.canet@irqsave.net> | 2014-02-13 17:22:33 +0100 |
|---|---|---|
| committer |   Stefan Hajnoczi <stefanha@redhat.com> | 2014-02-14 18:05:39 +0100 |
| commit | 57b6bdf37c64985cf02b8737c550d52759059c9d (patch) | |
| tree | 7af75dbbd25f396f1665f479fffb3397b57c9336 /hw/usb/dev-hub.c | |
| parent | block: mirror - use local_err to avoid NULL errp (diff) | |
| download | qemu-57b6bdf37c64985cf02b8737c550d52759059c9d.tar.xz qemu-57b6bdf37c64985cf02b8737c550d52759059c9d.zip | |
blockdev: Fix wrong usage of QDECREF causing snapshoted quorum to crash on close.
As bdrv_open() documentation states:
"The reference to the QDict belongs to the block layer
* after the call (even on failure), so if the caller intends to reuse the
* dictionary, it needs to use QINCREF() before calling bdrv_open."
the optional options dict will not be reused after bdrv_open() and should
belong to the block layer so remove the extra QDECREF(options).
Signed-off-by: Benoit Canet <benoit@irqsave.net>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'hw/usb/dev-hub.c')
0 files changed, 0 insertions, 0 deletions