From: Wootak Jung Date: Fri, 29 Oct 2021 05:25:24 +0000 (+0900) Subject: Fix memory leak X-Git-Tag: submit/tizen/20211102.014033^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d9c58ce718be9c0ee999adbf1c139a9fc309ae90;p=platform%2Fcore%2Ftelephony%2Ftel-plugin-dbus_tapi.git Fix memory leak Change-Id: I5e21b40b61185099a1d785c2172415144e379869 --- diff --git a/src/dtapi_sat.c b/src/dtapi_sat.c index 699542f..43f2889 100644 --- a/src/dtapi_sat.c +++ b/src/dtapi_sat.c @@ -324,6 +324,7 @@ static gboolean on_sat_envelope(TelephonySAT *sat, } req.envelop_data.raw_data.data_len = i; g_variant_iter_free(iter); + g_variant_unref(inner_gv); /* Dispatch request */ dtapi_dispatch_request(ctx, sat, invocation, diff --git a/src/dtapi_sim.c b/src/dtapi_sim.c index b2d1c96..cc26e4e 100644 --- a/src/dtapi_sim.c +++ b/src/dtapi_sim.c @@ -1635,7 +1635,7 @@ static gboolean on_sim_transfer_apdu(TelephonySim *sim, } req.apdu_length = (unsigned int)i; g_variant_iter_free(iter); - //g_variant_unref(inner_gv); + g_variant_unref(inner_gv); //g_variant_unref(apdu); tcore_util_hex_dump("[APDU_REQ] ", req.apdu_length, req.apdu); @@ -1678,7 +1678,7 @@ static gboolean on_sim_access_rsim_io(TelephonySim *sim, } req.data_len = (unsigned int)i; g_variant_iter_free(iter); - //g_variant_unref(inner_gv); + g_variant_unref(inner_gv); //g_variant_unref(rsimio); dbg("ef_id[%x], rec_index[%d]", req.field, req.rec_index); diff --git a/src/sat_ui_support/sat_ui_support.c b/src/sat_ui_support/sat_ui_support.c index 10b2b58..7a024f0 100644 --- a/src/sat_ui_support/sat_ui_support.c +++ b/src/sat_ui_support/sat_ui_support.c @@ -194,6 +194,7 @@ static gboolean _sat_ui_support_processing_setup_menu_ind(GVariant *data, char * local_index++; } g_variant_iter_free(iter); + g_variant_unref(unbox); } setup_menu.bIsSatMainMenuHelpInfo = (b_helpinfo ? 1 : 0); setup_menu.bIsUpdatedSatMainMenu = (b_updated ? 1 : 0); @@ -220,6 +221,7 @@ static gboolean _sat_ui_support_processing_setup_menu_ind(GVariant *data, char * setup_menu.iconId.iconInfo.height, setup_menu.iconId.iconInfo.ics, setup_menu.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } if (icon_list) { @@ -248,8 +250,10 @@ static gboolean _sat_ui_support_processing_setup_menu_ind(GVariant *data, char * icon_index++; } g_variant_iter_free(iter2); + g_variant_unref(unbox_list_info); } g_variant_iter_free(iter); + g_variant_unref(unbox_list); } #endif cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_SETUP_MENU); @@ -328,6 +332,7 @@ static gboolean _sat_ui_support_processing_display_text_ind(GVariant *data, char display_text.iconId.iconInfo.height, display_text.iconId.iconInfo.ics, display_text.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } #endif dbg("duration(%d) user_rsp(%d) immediately_rsp(%d)", duration, user_rsp_required, immediately_rsp); @@ -419,6 +424,7 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char local_index++; } g_variant_iter_free(iter); + g_variant_unref(unbox); } #if defined(TIZEN_SUPPORT_SAT_ICON) @@ -444,6 +450,7 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char select_item->iconId.iconInfo.height, select_item->iconId.iconInfo.ics, select_item->iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } if (icon_list) { @@ -472,8 +479,10 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char icon_index++; } g_variant_iter_free(iter2); + g_variant_unref(unbox_list_info); } g_variant_iter_free(iter); + g_variant_unref(unbox_list); } #endif cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_SELECT_ITEM); @@ -555,6 +564,7 @@ static gboolean _sat_ui_support_processing_get_inkey_ind(GVariant *data, char *s get_inkey.iconId.iconInfo.height, get_inkey.iconId.iconInfo.ics, get_inkey.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } #endif cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_GET_INKEY); @@ -638,6 +648,7 @@ static gboolean _sat_ui_support_processing_get_input_ind(GVariant *data, char *s get_input.iconId.iconInfo.height, get_input.iconId.iconInfo.ics, get_input.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } #endif cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_GET_INPUT); @@ -718,6 +729,7 @@ static gboolean _sat_ui_support_processing_refresh_ind(GVariant *data, char *slo refresh_info.iconId.iconInfo.height, refresh_info.iconId.iconInfo.ics, refresh_info.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } #endif cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_REFRESH); @@ -793,6 +805,7 @@ static gboolean _sat_ui_support_processing_play_tone_ind(GVariant *data, char *s play_tone_info.iconId.iconInfo.height, play_tone_info.iconId.iconInfo.ics, play_tone_info.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } #endif cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_PLAY_TONE); @@ -863,6 +876,7 @@ static gboolean _sat_ui_support_processing_idle_mode_text_ind(GVariant *data, ch idle_mode_text_info.iconId.iconInfo.height, idle_mode_text_info.iconId.iconInfo.ics, idle_mode_text_info.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } #endif cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_SETUP_IDLE_MODE_TEXT); @@ -944,6 +958,7 @@ static gboolean _sat_ui_support_processing_ui_info_ind(enum tel_sat_proactive_cm ui_info.iconId.iconInfo.height, ui_info.iconId.iconInfo.ics, ui_info.iconId.iconInfo.iconDataLen); } g_variant_iter_free(iter); + g_variant_unref(unbox); } #endif cmd_type = g_strdup_printf("%d", cmd); @@ -1253,6 +1268,7 @@ gboolean sat_ui_support_launch_eventdownloader_application(GVariant *data, enum local_index++; } g_variant_iter_free(iter); + g_variant_unref(unbox); } encoded_data = g_base64_encode((const guchar*)&eventinfo, sizeof(struct tel_sat_event_list));