afry-south / dragonfly-fcb

ÅF Dragonfly Quadrotor UAV Project
Other
7 stars 3 forks source link

vTaskList uses sprintf, not snprintf #130

Open adam-at-epsilon opened 8 years ago

adam-at-epsilon commented 8 years ago

vTaskList in FreeRTOS file tasks.c uses sprintfinstead of snprintf, this has the potential to create buffer overruns with many tasks.

This fcn is not called by FCB code at the time of writing.

adam-at-epsilon commented 8 years ago

latest FreeRTOS version has this problem too so submit to FreeRTOS.org as well.

adam-at-epsilon commented 8 years ago

Don't submit bug to FreeRTOS, they handle it by writing

This buffer is assumed to be large enough to contain the generated report. Approximately 40 bytes per task should be sufficient.

in task.h