This is an automated email from Gerrit.

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

-- gerrit

commit 733872bf23e5cd1e15b76099a92c81d26d273c00
Author: Tomas Vanek <[email protected]>
Date:   Wed Jan 9 13:01:50 2019 +0100

    target: do not allow 'target create' after init
    
    A target created after init lacks target_init_one() call
    and is not added to gdb targets.
    
    Steps to reproduce:
    - start OpenOCD with a dap target
    - connect by telnet
     target create ap0.mem mem_ap -dap $_CHIPNAME.dap -ap-num 0
     reset
    
    Segmentation fault is rised because target->check_reset is NULL.
    
    Change-Id: I2a62f3b450e4db3005c7041a22fb8f952e68c3b6
    Signed-off-by: Tomas Vanek <[email protected]>

diff --git a/src/target/target.c b/src/target/target.c
index b4bf5d3..7e30d78 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -5882,8 +5882,7 @@ static const struct command_registration 
target_subcommand_handlers[] = {
        },
        {
                .name = "create",
-               /* REVISIT this should be COMMAND_CONFIG ... */
-               .mode = COMMAND_ANY,
+               .mode = COMMAND_CONFIG,
                .jim_handler = jim_target_create,
                .usage = "name type '-chain-position' name [options ...]",
                .help = "Creates and selects a new target",

-- 


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

Reply via email to