This is an automated email from Gerrit.

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

-- gerrit

commit d05141cec91692d0806c1cfb195700d74e5c98e0
Author: Paul Fertser <[email protected]>
Date:   Fri Apr 24 14:43:45 2015 +0300

    rtos/mqx: prevent crash with -rtos auto
    
    Since mqx comes last in the list, with the auto option its
    update_threads is called even though it wasn't detected.
    
    This check should be removed from all the rtos helpers and moved to
    the generic code, but better do it later all in one go.
    
    Change-Id: If24ab42a58a468d90e9f12028d4c2fb76a9bc2e8
    Signed-off-by: Paul Fertser <[email protected]>

diff --git a/src/rtos/mqx.c b/src/rtos/mqx.c
index bbb96b2..b75311c 100644
--- a/src/rtos/mqx.c
+++ b/src/rtos/mqx.c
@@ -291,6 +291,9 @@ static int mqx_update_threads(
        uint16_t task_queue_size = 0;
        uint32_t active_td_addr = 0;
 
+       if (!rtos->symbols)
+               return -4;
+
        /* clear old data */
        rtos_free_threadlist(rtos);
        /* check scheduler */

-- 

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to