From 907c4910c3175ce3e9fc9bfb897527ca241ffd24 Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Sat, 31 Jan 2015 06:13:39 +0800 Subject: [PATCH] sim: Be sure of calling freeargv() after successfully call buildargv(). Or there will be memory leak. 2015-02-02 Chen Gang * microblaze/interp.c (sim_do_command): Call freeargv() before return. --- sim/ChangeLog | 5 +++++ sim/microblaze/interp.c | 3 +++ 2 files changed, 8 insertions(+) diff --git a/sim/ChangeLog b/sim/ChangeLog index 628e6a8b09f..a891ce51f7b 100644 --- a/sim/ChangeLog +++ b/sim/ChangeLog @@ -1,3 +1,8 @@ +2015-02-02 Chen Gang + + * microblaze/interp.c (sim_do_command): Call freeargv() before + return. + 2015-02-02 Chen Gang * mcore/interp.c (sim_do_command): Call freeargv() before return. diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c index 1c8a22d1543..4fc45952901 100644 --- a/sim/microblaze/interp.c +++ b/sim/microblaze/interp.c @@ -1019,6 +1019,7 @@ sim_do_command (SIM_DESC sd, const char *cmd) if ((simargv[1] == NULL) || (simargv[2] == NULL)) { fprintf (stderr, "Error: missing argument to watch cmd.\n"); + freeargv (simargv); return; } @@ -1062,6 +1063,8 @@ sim_do_command (SIM_DESC sd, const char *cmd) fprintf (stderr,"Error: \"%s\" is not a valid M.CORE simulator command.\n", cmd); } + + freeargv (simargv); } else { -- 2.30.2