sergiotarxz/nuttx
sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

sched: fix kasan report error when open CONFIG_DEBUG_MM

Browse Source 
reason:
1. g_running_tasks = thread A
2. thread A exit (free thread A's tcb) -> thread B
3. thread B interrupt by irq
4. check g_running_tasks->flags -> kasan report used after free

rootcause:
g_running_tasks has't set completely when syscall hanppened

Resolve:
Use rtcb (get at ISR begining) instead

Signed-off-by: ligd <liguiding1@xiaomi.com>
This commit is contained in:
ligd 2022-09-08 22:38:45 +08:00 committed by Petro Karashchenko
parent ed85055edb
commit 10529c7d0a
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
sched/irq/irq_dispatch.c
View File

@ -124,6 +124,9 @@
void irq_dispatch(int irq, FAR void *context)
{
#ifdef CONFIG_DEBUG_MM
struct tcb_s *rtcb = this_task();
#endif
xcpt_t vector = irq_unexpected_isr;
FAR void *arg = NULL;
unsigned int ndx = irq;
@ -175,7 +178,7 @@ void irq_dispatch(int irq, FAR void *context)
#endif
#ifdef CONFIG_DEBUG_MM
if ((g_running_tasks[this_cpu()]->flags & TCB_FLAG_HEAPCHECK) || \
if ((rtcb->flags & TCB_FLAG_HEAPCHECK) || \
(this_task()->flags & TCB_FLAG_HEAPCHECK))
{
kmm_checkcorruption();