Fix memory leak 07/265807/1 submit/tizen/20211102.014033
authorWootak Jung <wootak.jung@samsung.com>
Fri, 29 Oct 2021 05:25:24 +0000 (14:25 +0900)
committerWootak Jung <wootak.jung@samsung.com>
Fri, 29 Oct 2021 05:26:06 +0000 (14:26 +0900)
Change-Id: I5e21b40b61185099a1d785c2172415144e379869

src/dtapi_sat.c
src/dtapi_sim.c
src/sat_ui_support/sat_ui_support.c

index 699542f76e26f7ae6f98c6cafea750736b88117b..43f2889d7625b8fa4d7466e67229c64f8a4d8631 100644 (file)
@@ -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,
index b2d1c96aea5b755ee0b905a7d15223d505871893..cc26e4e54d6bfbd9138f288338c9ad211f18bf13 100644 (file)
@@ -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);
index 10b2b58de7551d5816003005ba859ccb01222cc2..7a024f094c74d9d1c944fa9638e0db8455d3252b 100644 (file)
@@ -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));