diff options
author | Zenghui Yu <zenghui.yu@linux.dev> | 2025-07-15 00:01:38 +0800 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2025-07-21 11:19:56 +0100 |
commit | f19310b23a00b5c19f930e4d57fc298744d11740 (patch) | |
tree | ef4d383228899a45ca93fa21cffc3164a8090a48 /scripts/tracetool/backend/log.py | |
parent | 082933a1f7d3c8e4a9e999c3d284928ef866c67d (diff) | |
download | qemu-f19310b23a00b5c19f930e4d57fc298744d11740.zip qemu-f19310b23a00b5c19f930e4d57fc298744d11740.tar.gz qemu-f19310b23a00b5c19f930e4d57fc298744d11740.tar.bz2 |
hvf: arm: Add permission check in GIC sysreg handlers
Quoting Peter Maydell:
" hvf_sysreg_read_cp() and hvf_sysreg_write_cp() do not check the .access
field of the ARMCPRegInfo to ensure that they forbid writes to registers
that are marked with a .access field that says they're read-only (and
ditto reads to write-only registers). "
Before we add more registers in GIC sysreg handlers, let's get it correct
by adding the .access checks to hvf_sysreg_read_cp() and
hvf_sysreg_write_cp(). With that, a sysreg access with invalid permission
will result in an UNDEFINED exception.
Suggested-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Zenghui Yu <zenghui.yu@linux.dev>
Message-id: 20250714160139.10404-2-zenghui.yu@linux.dev
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'scripts/tracetool/backend/log.py')
0 files changed, 0 insertions, 0 deletions