[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] memory: Forbid memory_region_set_address() on AS root
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH] memory: Forbid memory_region_set_address() on AS root |
Date: |
Sat, 17 Apr 2021 14:59:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 |
On 4/17/21 2:53 PM, Philippe Mathieu-Daudé wrote:
> To be sure an AddressSpace is zero-based, forbid changing
> base address of MemoryRegion used as AddressSpace container.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> Based-on: <20210417103028.601124-1-f4bug@amsat.org>
> ---
> softmmu/memory.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/softmmu/memory.c b/softmmu/memory.c
> index 409bcaec7f5..73c0e6f84f5 100644
> --- a/softmmu/memory.c
> +++ b/softmmu/memory.c
> @@ -2522,6 +2522,7 @@ static void memory_region_readd_subregion(MemoryRegion
> *mr)
>
> void memory_region_set_address(MemoryRegion *mr, hwaddr addr)
> {
> + assert(!mr->is_root_mr);
Hmm to not discriminate the API usage, maybe safer as:
assert(!(addr && mr->is_root_mr));
> if (addr != mr->addr) {
> mr->addr = addr;
> memory_region_readd_subregion(mr);
>