This is an automated email from Gerrit.

Steven Stallion ([email protected]) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/4719

-- gerrit

commit c0342ed17006e683eabd09ab7a64ea38cd897a41
Author: Steven Stallion <[email protected]>
Date:   Tue Oct 9 13:48:18 2018 -0500

    rtos: check symbol list when updating uCOS-III
    
    This patch corrects a crash in uCOS-III on a new GDB connection when
    RTOS autodetection is not used. The crash was caused by not checking if
    the symbol list had been loaded prior to updating threads.
    
    Change-Id: I64c5133e02fe22fc8d14584cc40d87b49c935b0b
    Signed-off-by: Steven Stallion <[email protected]>

diff --git a/src/rtos/uCOS-III.c b/src/rtos/uCOS-III.c
index 8e63ea4..8d84365 100644
--- a/src/rtos/uCOS-III.c
+++ b/src/rtos/uCOS-III.c
@@ -286,6 +286,11 @@ static int uCOS_III_update_threads(struct rtos *rtos)
        struct uCOS_III_params *params = rtos->rtos_specific_params;
        int retval;
 
+       if (rtos->symbols == NULL) {
+               LOG_ERROR("uCOS-III: symbol list not loaded");
+               return ERROR_FAIL;
+       }
+
        /* free previous thread details */
        rtos_free_threadlist(rtos);
 

-- 


_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to