Stack protection needs improvements

embox / embox

Modular and configurable OS for embedded applications

http://embox.github.io

BSD 2-Clause "Simplified" License

1.17k stars 276 forks source link

Open AntonKozlov opened 7 years ago

AntonKozlov commented 7 years ago

Disabled. Should be enabled by default
Seems to be disabled for anything except stack_protect_test: stack_protect_enable called in setup and stack_protect_disable in teardown
Should not require setting fault_handler. SIGSEGV signal looks more appropriate, but questionable.
Test should actually check that access within stack is OK, now it's just // should be OK
Stack protection could be armed/disarmed in context switch, this will reduce number of entries in process vmem context

puranjaymohan commented 4 years ago

I would like to work on this. Please give me a few recommendations where I can start. Thanks