interesting
/
riscv-openocd
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

rtos: chibios: replace malloc+sprintf with alloc_printf 

This makes it safer and simpler at the same time.

Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Change-Id: Ie294f1f6033ffc9f46b39210e2f7fc2f648e80ac
Reviewed-on: https://review.openocd.org/c/openocd/+/8598
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
This commit is contained in:
Paul Fertser 2024-11-26 23:16:15 +02:00 committed by Tomas Vanek
parent a510d51a78
commit ce38758e3d
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/rtos/chibios.c
View File

@ -421,9 +421,11 @@ static int chibios_update_threads(struct rtos *rtos)
else else
state_desc = "Unknown"; state_desc = "Unknown";
curr_thrd_details->extra_info_str = malloc(strlen( curr_thrd_details->extra_info_str = alloc_printf("State: %s", state_desc);
state_desc)+8); if (!curr_thrd_details->extra_info_str) {
sprintf(curr_thrd_details->extra_info_str, "State: %s", state_desc); LOG_ERROR("Could not allocate space for thread state description");
return -1;
}
curr_thrd_details->exists = true; curr_thrd_details->exists = true;