[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 9/9] memory: assert MemoryRegionOps callbacks are defined
From: |
P J P |
Subject: |
[PATCH v3 9/9] memory: assert MemoryRegionOps callbacks are defined |
Date: |
Tue, 30 Jun 2020 17:57:10 +0530 |
From: Prasad J Pandit <pjp@fedoraproject.org>
When registering a MemoryRegionOps object, assert that its
read/write callback methods are defined. This avoids potential
guest crash via a NULL pointer dereference.
Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Li Qiang <liq3ea@gmail.com>
Signed-off-by: Prasad J Pandit <pjp@fedoraproject.org>
---
memory.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
Update v3: Add Reviewed-by: ...
-> https://lists.gnu.org/archive/html/qemu-devel/2020-06/msg09467.html
diff --git a/memory.c b/memory.c
index 9200b20130..5e22bce326 100644
--- a/memory.c
+++ b/memory.c
@@ -1485,7 +1485,13 @@ void memory_region_init_io(MemoryRegion *mr,
uint64_t size)
{
memory_region_init(mr, owner, name, size);
- mr->ops = ops ? ops : &unassigned_mem_ops;
+ if (ops) {
+ assert(ops->read || ops->read_with_attrs);
+ assert(ops->write || ops->write_with_attrs);
+ mr->ops = ops;
+ } else {
+ mr->ops = &unassigned_mem_ops;
+ }
mr->opaque = opaque;
mr->terminates = true;
}
@@ -1663,6 +1669,8 @@ void memory_region_init_rom_device_nomigrate(MemoryRegion
*mr,
{
Error *err = NULL;
assert(ops);
+ assert(ops->read || ops->read_with_attrs);
+ assert(ops->write || ops->write_with_attrs);
memory_region_init(mr, owner, name, size);
mr->ops = ops;
mr->opaque = opaque;
--
2.26.2
- [PATCH v3 0/9] memory: assert and define MemoryRegionOps callbacks, P J P, 2020/06/30
- [PATCH v3 1/9] hw/pci-host: add pci-intack write method, P J P, 2020/06/30
- [PATCH v3 2/9] pci-host: add pcie-msi read method, P J P, 2020/06/30
- [PATCH v3 3/9] vfio: add quirk device write method, P J P, 2020/06/30
- [PATCH v3 4/9] prep: add ppc-parity write method, P J P, 2020/06/30
- [PATCH v3 5/9] nvram: add nrf51_soc flash read method, P J P, 2020/06/30
- [PATCH v3 6/9] spapr_pci: add spapr msi read method, P J P, 2020/06/30
- [PATCH v3 7/9] tz-ppc: add dummy read/write methods, P J P, 2020/06/30
- [PATCH v3 8/9] imx7-ccm: add digprog mmio write method, P J P, 2020/06/30
- [PATCH v3 9/9] memory: assert MemoryRegionOps callbacks are defined,
P J P <=
- Re: [PATCH v3 0/9] memory: assert and define MemoryRegionOps callbacks, no-reply, 2020/06/30