diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2020-03-23 17:22:30 +0000 |
|---|---|---|
| committer | Peter Maydell <peter.maydell@linaro.org> | 2020-03-23 17:22:30 +0000 |
| commit | ae1111d4def40c6f592c3a307c599272b778eb65 (patch) | |
| tree | bd47a6ea68c5544cbb1c5f0260c374bda6cbcf62 /qemu-io.c | |
| parent | 4dabf39592e92d692c6f2a1633571114ae25d843 (diff) | |
| download | qemu-ae1111d4def40c6f592c3a307c599272b778eb65.zip qemu-ae1111d4def40c6f592c3a307c599272b778eb65.tar.gz qemu-ae1111d4def40c6f592c3a307c599272b778eb65.tar.bz2 | |
target/arm: Rearrange disabled check for watchpoints
Coverity rightly notes that ctz32(bas) on 0 will return 32,
which makes the len calculation a BAD_SHIFT.
A value of 0 in DBGWCR<n>_EL1.BAS is reserved. Simply move
the existing check we have for this case.
Reported-by: Coverity (CID 1421964)
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-id: 20200320160622.8040-2-richard.henderson@linaro.org
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'qemu-io.c')
0 files changed, 0 insertions, 0 deletions