Deallocate memory before setting new value

- Issue:
Memory leak can be possible if vc_cmd_db module sets the new value in
vc_cmd_s structure.

- Solution:
The possibility of the issue is because the module did not check the
preivous value of the member. Thus, this patch adds code for checking
the member of the structure. If there is already value set, the module
will release the old value and then set new value.

Change-Id: I2d2676d08a4cacb75c32983c1a49eced28371b65
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

diff --git a/common/vc_cmd_db.c b/common/vc_cmd_db.c
index e41b291481dabeeafdee0eeda916076e4c9c466c..154a4fa77300f58f7b226d8cdf3508415cd659c4 100644 (file)
--- a/common/vc_cmd_db.c
+++ b/common/vc_cmd_db.c
@@ -2185,6 +2185,8 @@ static int __vc_db_insert_command(sqlite3* db_handle, int pid, vc_cmd_type_e typ
                                        tmp_cmd->command = NULL;
                                }
                                tmp_cmd->command = strdup(temp_command);
+
+                               free(tmp_cmd->fixed);
                                if (NULL != fixed_cmd)
                                        tmp_cmd->fixed = strdup(fixed_cmd);
                                else