diff options
| author | Richard Henderson | 2022-06-21 17:38:29 +0200 |
|---|---|---|
| committer | Richard Henderson | 2022-06-21 18:26:16 +0200 |
| commit | 418ade7849ce7641c0f7333718caf5091a02fd4c (patch) | |
| tree | be8c04dd9ffd4ec5ed0f367c815c89bd23c66989 /util/cacheflush.c | |
| parent | qemu-timer: Skip empty timer lists before locking in qemu_clock_deadline_ns_all (diff) | |
| download | qemu-418ade7849ce7641c0f7333718caf5091a02fd4c.tar.gz qemu-418ade7849ce7641c0f7333718caf5091a02fd4c.tar.xz qemu-418ade7849ce7641c0f7333718caf5091a02fd4c.zip | |
softmmu: Always initialize xlat in address_space_translate_for_iotlb
The bug is an uninitialized memory read, along the translate_fail
path, which results in garbage being read from iotlb_to_section,
which can lead to a crash in io_readx/io_writex.
The bug may be fixed by writing any value with zero
in ~TARGET_PAGE_MASK, so that the call to iotlb_to_section using
the xlat'ed address returns io_mem_unassigned, as desired by the
translate_fail path.
It is most useful to record the original physical page address,
which will eventually be logged by memory_region_access_valid
when the access is rejected by unassigned_mem_accepts.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1065
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20220621153829.366423-1-richard.henderson@linaro.org>
Diffstat (limited to 'util/cacheflush.c')
0 files changed, 0 insertions, 0 deletions