[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] target/xtensa: Assert that interrupt level is within bounds
From: |
Max Filippov |
Subject: |
Re: [PATCH] target/xtensa: Assert that interrupt level is within bounds |
Date: |
Fri, 23 Jun 2023 17:20:22 -0700 |
On Fri, Jun 23, 2023 at 8:41 AM Peter Maydell <peter.maydell@linaro.org> wrote:
>
> 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>
> ---
> NB: only tested with 'make check-avocado'. You could argue that we
> should mark the coverity issue as false-positive instead if you like.
> ---
> target/xtensa/exc_helper.c | 3 +++
> 1 file changed, 3 insertions(+)
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
--
Thanks.
-- Max