Revert "Fix the memory leak in GVariant use" 70/117870/1
authorjeesun kim <iamjs.kim@samsung.com>
Wed, 8 Mar 2017 01:39:36 +0000 (17:39 -0800)
committerjeesun kim <iamjs.kim@samsung.com>
Wed, 8 Mar 2017 01:39:36 +0000 (17:39 -0800)
This reverts commit c0356d199ff5f946bf7f77dd59a4fd7491703217.

Change-Id: I88ed1373fc8f7b9a4cd6b9df365590198c1eadae

common/dbus/cal_dbus_helper.c

index c84eb68..389f79b 100644 (file)
@@ -1723,8 +1723,6 @@ int cal_dbus_utils_gvariant_to_record(GVariant *arg_record_pack, calendar_record
                return CALENDAR_ERROR_INVALID_PARAMETER;
 
        _gvariant_to_record(type, arg_common, arg_record, out_record);
-       g_variant_unref(arg_common);
-       g_variant_unref(arg_record);
        return CALENDAR_ERROR_NONE;
 }
 
@@ -1746,19 +1744,14 @@ int cal_dbus_utils_gvariant_to_list(GVariant *arg_list_pack, calendar_list_h *ou
                        calendar_record_h record = NULL;
                        int ret = 0;
                        ret = cal_dbus_utils_gvariant_to_record(arg_record, &record);
-                       if (CALENDAR_ERROR_NONE != ret) {
-                               g_variant_unref(arg_record);
+                       if (CALENDAR_ERROR_NONE != ret)
                                break;
-                       }
 
                        calendar_list_add(l, record);
-                       g_variant_unref(arg_record);
                }
-               g_variant_iter_free(iter_value);
        }
 
        *out_list = l;
-       g_variant_unref(arg_list);
 
        return CALENDAR_ERROR_NONE;
 }