Skip to content

Commit

Permalink
target/xtensa: Assert that interrupt level is within bounds
Browse files Browse the repository at this point in the history
In handle_interrupt() we use level as an index into the interrupt_vector[]
array. This is safe because we have checked it against env->config->nlevel,
but Coverity can't see that (and it is only true because each CPU config
sets its XCHAL_NUM_INTLEVELS to something less than MAX_NLEVELS), so it
complains about a possible array overrun (CID 1507131)

Add an assert() which will make Coverity happy and catch the unlikely
case of a mis-set XCHAL_NUM_INTLEVELS in future.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
Message-id: 20230623154135.1930261-1-peter.maydell@linaro.org
  • Loading branch information
pm215 committed Jul 4, 2023
1 parent 7812aaa commit 86a7827
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions target/xtensa/exc_helper.c
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,9 @@ static void handle_interrupt(CPUXtensaState *env)
CPUState *cs = env_cpu(env);

if (level > 1) {
/* env->config->nlevel check should have ensured this */
assert(level < sizeof(env->config->interrupt_vector));

env->sregs[EPC1 + level - 1] = env->pc;
env->sregs[EPS2 + level - 2] = env->sregs[PS];
env->sregs[PS] =
Expand Down

0 comments on commit 86a7827

Please sign in to comment.