diff options
| author | Xiaojuan Yang | 2022-07-15 08:07:40 +0200 |
|---|---|---|
| committer | Richard Henderson | 2022-07-19 18:23:58 +0200 |
| commit | 064357041d64331e3dbc18629d07abf88ce0f6bb (patch) | |
| tree | 05521aef2d90f4526413a9fa4e7a091b12f7df00 /target/loongarch | |
| parent | target/loongarch/tlb_helper: Fix coverity integer overflow error (diff) | |
| download | qemu-064357041d64331e3dbc18629d07abf88ce0f6bb.tar.gz qemu-064357041d64331e3dbc18629d07abf88ce0f6bb.tar.xz qemu-064357041d64331e3dbc18629d07abf88ce0f6bb.zip | |
target/loongarch/op_helper: Fix coverity cond_at_most error
The boundary size of cpucfg array should be 0 to ARRAY_SIZE(cpucfg)-1.
So, using index bigger than max boundary to access cpucfg[] must be
forbidden.
Fix coverity CID: 1489760
Signed-off-by: Xiaojuan Yang <yangxiaojuan@loongson.cn>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220715060740.1500628-6-yangxiaojuan@loongson.cn>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'target/loongarch')
| -rw-r--r-- | target/loongarch/op_helper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target/loongarch/op_helper.c b/target/loongarch/op_helper.c index 4b429b6699..568c071601 100644 --- a/target/loongarch/op_helper.c +++ b/target/loongarch/op_helper.c @@ -81,7 +81,7 @@ target_ulong helper_crc32c(target_ulong val, target_ulong m, uint64_t sz) target_ulong helper_cpucfg(CPULoongArchState *env, target_ulong rj) { - return rj > 21 ? 0 : env->cpucfg[rj]; + return rj >= ARRAY_SIZE(env->cpucfg) ? 0 : env->cpucfg[rj]; } uint64_t helper_rdtime_d(CPULoongArchState *env) |