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 628e6a8..a891ce5 100644 --- a/sim/ChangeLog +++ b/sim/ChangeLog @@ -1,5 +1,10 @@ 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. 2015-02-02 Chen Gang diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c index 1c8a22d..4fc4595 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.7.4