umts / embedded-departure-board

A Zephyr based departure board that frequently GETs & displays bus stop departure times over LTE-M
Apache License 2.0
2 stars 0 forks source link

Local Stack Overflow in `update_stop()` #37

Closed bandogora closed 2 months ago

bandogora commented 4 months ago

A local stack overflow is caught by the stack canary on the return of the update_stop() function. This occasionally crashes the app. The watchdog keeps things working, but this is a security issue and needs to be fixed before deployment.

https://github.com/umts/embedded-departure-board/blob/f156f1c13b5ebc712b87f93043e3f0ccb17e3e8e/app/src/update_stop.c#L117-L121