diff options
author | Marc Schink <openocd-dev@marcschink.de> | 2018-04-04 09:50:40 +0200 |
---|---|---|
committer | Tomas Vanek <vanekt@fbl.cz> | 2018-07-18 21:27:12 +0100 |
commit | abc270c8837bc7aad3f390e4e9858e4f652261b7 (patch) | |
tree | 71a21921c2ce9e90ea8b0dcfcdc0455801b32ab0 /src | |
parent | d293583e03b2615aaee1cae21796771c87b48ee2 (diff) |
target: Fix segfault for 'mem2array'
Call 'mem2array' without arguments to reproduce the segmentation
fault.
Change-Id: I02bf46cc8bd317abbb721a8c75d7cbfac99eb34e
Signed-off-by: Marc Schink <openocd-dev@marcschink.de>
Reviewed-on: http://openocd.zylin.com/4534
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-by: Christopher Head <chead@zaber.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/target/target.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/target/target.c b/src/target/target.c index 1ca6c8e5..591b9ea2 100644 --- a/src/target/target.c +++ b/src/target/target.c @@ -4131,8 +4131,9 @@ static int target_mem2array(Jim_Interp *interp, struct target *target, int argc, * argv[3] = memory address * argv[4] = count of times to read */ + if (argc < 4 || argc > 5) { - Jim_WrongNumArgs(interp, 1, argv, "varname width addr nelems [phys]"); + Jim_WrongNumArgs(interp, 0, argv, "varname width addr nelems [phys]"); return JIM_ERR; } varname = Jim_GetString(argv[0], &len); |