jlaitine opened a new pull request, #16804:
URL: https://github.com/apache/nuttx/pull/16804

   ## Summary
   
   After the signals have been delivered, the local irqs need to be disabled 
until the context switch. But just calling leave_critical_section(regs[REG_PS]) 
will enable them if they were enabled in the context.
   
   Any interrupt on this cpu before xtensa_context_restore() would set the 
rtcb->xcp.regs to NULL, causing a crash in context restore.
   
   Fix this by calling leave_critical_section(up_irq_save()) instead.
   
   ## Impact
   
   Fixes a random crash during signal delivery in esp32
   
   ## Testing
   
   Tested on esp32 qemu
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to