Skip to content
This repository has been archived by the owner on May 3, 2022. It is now read-only.

vTaskList uses sprintf, not snprintf #130

Open
adam-at-epsilon opened this issue Sep 28, 2015 · 2 comments
Open

vTaskList uses sprintf, not snprintf #130

adam-at-epsilon opened this issue Sep 28, 2015 · 2 comments
Labels
bug nicetohave

Comments

@adam-at-epsilon
Copy link
Contributor

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
Copy link
Contributor Author

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

@adam-at-epsilon
Copy link
Contributor Author

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug nicetohave
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@adam-at-epsilon