It is useless to assert that block is not NULL because it is already dereferenced in the first line of the function.
We could split the declaration and initialization of oldsize, but then we would need to remove the const qualifier. This seems worse, as the assertion would be almost useless anyway. Signed-off-by: Sergei Heifetz <[email protected]> --- system/physmem.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/system/physmem.c b/system/physmem.c index b0311f4531..317b359ebe 100644 --- a/system/physmem.c +++ b/system/physmem.c @@ -2057,8 +2057,6 @@ int qemu_ram_resize(RAMBlock *block, ram_addr_t newsize, Error **errp) const ram_addr_t oldsize = block->used_length; const ram_addr_t unaligned_size = newsize; - assert(block); - newsize = TARGET_PAGE_ALIGN(newsize); newsize = REAL_HOST_PAGE_ALIGN(newsize); -- 2.34.1
