commit fecc0dbf13aa6f246c6139f3255088974001784f
author Eugene Tarassov <eugene.tarassov@windriver.com> Wed Jun 06 10:40:24 2012 -0700
committer Eugene Tarassov <eugene.tarassov@windriver.com> Thu Jun 07 17:25:29 2012 -0700
tree d2b8071fa0440cc5bbcdeeac9f5551282b9b5b91
parent 3989f288dd1c08263cd99e6b68b8d53221447422

TCF Agent: fixed a memory leak

agent/tcf/services/symbols_proxy.c

diff --git a/agent/tcf/services/symbols_proxy.c b/agent/tcf/services/symbols_proxy.c
index 4daa203..1c37326 100644
--- a/agent/tcf/services/symbols_proxy.c
+++ b/agent/tcf/services/symbols_proxy.c
@@ -294,13 +294,15 @@
     }
 }
 
+static void free_location_command_args(LocationExpressionCommand * cmd) {
+    if (cmd->cmd == SFT_CMD_LOCATION) loc_free(cmd->args.loc.code_addr);
+    else if (cmd->cmd == SFT_CMD_PIECE) loc_free(cmd->args.piece.value);
+}
+
 static void free_sft_sequence(StackFrameRegisterLocation * seq) {
     if (seq != NULL) {
         unsigned i = 0;
-        while (i < seq->cmds_cnt) {
-            LocationExpressionCommand * cmd = seq->cmds + i++;
-            if (cmd->cmd == SFT_CMD_LOCATION) loc_free(cmd->args.loc.code_addr);
-        }
+        while (i < seq->cmds_cnt) free_location_command_args(seq->cmds + i++);
         loc_free(seq);
     }
 }
@@ -322,10 +324,7 @@
 
 static void free_location_commands(LocationCommands * cmds) {
     unsigned i = 0;
-    while (i < cmds->cnt) {
-        LocationExpressionCommand * cmd = cmds->cmds + i++;
-        if (cmd->cmd == SFT_CMD_LOCATION) loc_free(cmd->args.loc.code_addr);
-    }
+    while (i < cmds->cnt) free_location_command_args(cmds->cmds + i++);
     loc_free(cmds->cmds);
 }