From: OpenOCD-Gerrit <ope...@us...> - 2016-05-04 21:40:13
|
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Main OpenOCD repository". The branch, master has been updated via 0f65e36ee176777a62fa0bbb8c31491928e98cf2 (commit) from f5b7033742e0858610d37808f15b25602ba62995 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 0f65e36ee176777a62fa0bbb8c31491928e98cf2 Author: Jonathan Dumaresq <jdu...@ci...> Date: Fri Apr 8 10:38:00 2016 -0400 RTOS support: Correction of out of bound access of thread array FreeRTOS use an array to store ready task. The array size is configMAX_PRIORITIES. In the current implementation, the code try to access 1 more priority if the helper from freeRTOS contrib is used. This has effect of detecting bad thread. This patch correct this and have been tested on a code with more than 12 task. Change-Id: Id229f0b2c4bf1aab87a2a69be174cc9b6dda00cb Signed-off-by: Jonathan Dumaresq <jdu...@ci...> Reviewed-on: http://openocd.zylin.com/3400 Tested-by: jenkins Reviewed-by: Tomas Vanek <va...@fb...> diff --git a/contrib/rtos-helpers/FreeRTOS-openocd.c b/contrib/rtos-helpers/FreeRTOS-openocd.c index 567e710..81a3ab7 100644 --- a/contrib/rtos-helpers/FreeRTOS-openocd.c +++ b/contrib/rtos-helpers/FreeRTOS-openocd.c @@ -17,4 +17,4 @@ #define USED #endif -const int USED uxTopUsedPriority = configMAX_PRIORITIES; +const int USED uxTopUsedPriority = configMAX_PRIORITIES - 1; ----------------------------------------------------------------------- Summary of changes: contrib/rtos-helpers/FreeRTOS-openocd.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) hooks/post-receive -- Main OpenOCD repository |