c4dab45526
- qga/win32: Use rundll for VSS installation - qga/win32: Remove change action from MSI installer - ide: Increment BB in-flight counter for TRIM BH - hw/pci-bridge/pxb: Fix missing swizzle - host-vdpa: make notifiers _init()/_uninit() symmetric - hw/virtio: vdpa: Fix leak of host-notifier memory-region - accel/tcg/cpu-exec: Fix precise single-stepping after interrupt - Allow setting up to 8 bytes with the generic loader - hw/net/virtio-net: make some VirtIONet const - accel/tcg: Optimize jump cache flush during tlb range flush - 9pfs: prevent opening special files (CVE-2023-2861) - tcg: Reduce tcg_assert_listed_vecop() scope - gitlab: Disable plugins for cross-i386-tci - vfio/pci: Fix a segfault in vfio_realize - block/iscsi: fix double-free on BUSY or similar statuses - tests/tcg: fix unused variable in linux-test - hw/net/vmxnet3: allow VMXNET3_MAX_MTU itself as a value - qga/vss-win32: fix warning for clang++-15 - vnc: avoid underflow when accessing user-provided address - block/monitor: Fix crash when executing HMP commit - virtio-gpu: add a FIXME for virtio_gpu_load() - hw/ppc/Kconfig: MAC_NEWWORLD should always select USB_OHCI_PCI - migration: report compress thread pid to libvirt Signed-off-by: Jiabo Feng <fengjiabo1@huawei.com>
55 lines
2.0 KiB
Diff
55 lines
2.0 KiB
Diff
From 33dfb9d81a8cfe17aaa3f0804cbd491b06d38cd6 Mon Sep 17 00:00:00 2001
|
|
From: qihao <qihao_yewu@cmss.chinamobile.com>
|
|
Date: Tue, 27 Jun 2023 14:40:13 +0800
|
|
Subject: [PATCH] block/monitor: Fix crash when executing HMP commit
|
|
|
|
cheery-pick from b7b814cd87a5fbe9f0fb5732dd28932699317bda
|
|
|
|
hmp_commit() calls blk_is_available() from a non-coroutine context (and
|
|
in the main loop). blk_is_available() is a co_wrapper_mixed_bdrv_rdlock
|
|
function, and in the non-coroutine context it calls AIO_WAIT_WHILE(),
|
|
which crashes if the aio_context lock is not taken before.
|
|
|
|
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1615
|
|
Signed-off-by: Wang Liang <wangliangzz@inspur.com>
|
|
Message-Id: <20230424103902.45265-1-wangliangzz@126.com>
|
|
Reviewed-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
|
|
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
|
|
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
(cherry picked from commit 8c1e8fb2e7fc2cbeb57703e143965a4cd3ad301a)
|
|
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
|
|
Signed-off-by: qihao_yewu <qihao_yewu@cmss.chinamobile.com>
|
|
---
|
|
block/monitor/block-hmp-cmds.c | 10 ++++++----
|
|
1 file changed, 6 insertions(+), 4 deletions(-)
|
|
|
|
diff --git a/block/monitor/block-hmp-cmds.c b/block/monitor/block-hmp-cmds.c
|
|
index 2ac4aedfff..44f0af3430 100644
|
|
--- a/block/monitor/block-hmp-cmds.c
|
|
+++ b/block/monitor/block-hmp-cmds.c
|
|
@@ -213,15 +213,17 @@ void hmp_commit(Monitor *mon, const QDict *qdict)
|
|
error_report("Device '%s' not found", device);
|
|
return;
|
|
}
|
|
- if (!blk_is_available(blk)) {
|
|
- error_report("Device '%s' has no medium", device);
|
|
- return;
|
|
- }
|
|
|
|
bs = bdrv_skip_implicit_filters(blk_bs(blk));
|
|
aio_context = bdrv_get_aio_context(bs);
|
|
aio_context_acquire(aio_context);
|
|
|
|
+ if (!blk_is_available(blk)) {
|
|
+ error_report("Device '%s' has no medium", device);
|
|
+ aio_context_release(aio_context);
|
|
+ return;
|
|
+ }
|
|
+
|
|
ret = bdrv_commit(bs);
|
|
|
|
aio_context_release(aio_context);
|
|
--
|
|
2.41.0.windows.1
|
|
|