diff options
| author | Peter Maydell <peter.maydell@linaro.org> | 2024-07-23 18:05:13 +0100 |
|---|---|---|
| committer | Peter Maydell <peter.maydell@linaro.org> | 2024-07-29 17:03:35 +0100 |
| commit | 73188068d7ba40c8a37b4763db38bb1ce24ca07d (patch) | |
| tree | 35e0fa8ae2dceec60567b763f6cdf3c6ccc0b2e1 /hw/net/cadence_gem.c | |
| parent | 525650cd71104c046e4260b5acaeb275f520d5c0 (diff) | |
| download | focaccia-qemu-73188068d7ba40c8a37b4763db38bb1ce24ca07d.tar.gz focaccia-qemu-73188068d7ba40c8a37b4763db38bb1ce24ca07d.zip | |
system/physmem: Where we assume we have a RAM MR, assert it
In the functions invalidate_and_set_dirty() and cpu_physical_memory_snapshot_and_clear_dirty(), we assume that we are dealing with RAM memory regions. In this case we know that memory_region_get_ram_addr() will succeed. Assert this before we use the returned ram_addr_t in arithmetic. This makes Coverity happier about these functions: it otherwise complains that we might have an arithmetic overflow that stems from the possible -1 return from memory_region_get_ram_addr(). Resolves: Coverity CID 1547629, 1547715 Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Peter Xu <peterx@redhat.com> Reviewed-by: David Hildenbrand <david@redhat.com> Message-id: 20240723170513.1676453-1-peter.maydell@linaro.org
Diffstat (limited to 'hw/net/cadence_gem.c')
0 files changed, 0 insertions, 0 deletions