adding null check for preventing crash 06/221306/2 accepted/tizen/unified/20200102.220730 submit/tizen/20200102.085236
authorAbhishek Vijay <abhishek.v@samsung.com>
Tue, 31 Dec 2019 04:53:15 +0000 (10:23 +0530)
committerAbhishek Vijay <abhishek.v@samsung.com>
Tue, 31 Dec 2019 08:24:34 +0000 (13:54 +0530)
Change-Id: I7956a303914ce7f21923dea3a2f013c738948639
Signed-off-by: Abhishek Vijay <abhishek.v@samsung.com>
client/cal_client_dbus.c

index 30fb0a4..5362a7e 100644 (file)
@@ -447,7 +447,8 @@ int cal_dbus_insert_records(calendar_h handle, calendar_list_h list,
                g_error_free(error);
                g_variant_unref(arg_list);
                g_variant_unref(arg_handle);
-               g_variant_unref(arg_ids);
+               if (arg_ids != NULL)
+                       g_variant_unref(arg_ids);
                return ret;
                /* LCOV_EXCL_STOP */
        }
@@ -595,7 +596,8 @@ int cal_dbus_get_record(calendar_h handle, const char *view_uri, int id,
                        ret = CALENDAR_ERROR_IPC;
                g_error_free(error);
                g_variant_unref(arg_handle);
-               g_variant_unref(arg_record);
+               if (arg_record != NULL)
+                       g_variant_unref(arg_record);
                return ret;
                /* LCOV_EXCL_STOP */
        }
@@ -627,7 +629,8 @@ int cal_dbus_get_all_records(calendar_h handle, const char *view_uri,
                        ret = CALENDAR_ERROR_IPC;
                g_error_free(error);
                g_variant_unref(arg_handle);
-               g_variant_unref(arg_list);
+               if (arg_list != NULL)
+                       g_variant_unref(arg_list);
                return ret;
                /* LCOV_EXCL_STOP */
        }
@@ -662,7 +665,8 @@ int cal_dbus_get_records_with_query(calendar_h handle, calendar_query_h query,
                else
                        ret = CALENDAR_ERROR_IPC;
                g_error_free(error);
-               g_variant_unref(arg_list);
+               if (arg_list != NULL)
+                       g_variant_unref(arg_list);
                return ret;
                /* LCOV_EXCL_STOP */
        }
@@ -864,7 +868,8 @@ int cal_dbus_get_changes_by_version(calendar_h handle, const char *view_uri,
                        ret = CALENDAR_ERROR_IPC;
                g_error_free(error);
                g_variant_unref(arg_handle);
-               g_variant_unref(arg_list);
+               if (arg_list != NULL)
+                       g_variant_unref(arg_list);
                return ret;
                /* LCOV_EXCL_STOP */
        }
@@ -900,7 +905,8 @@ int cal_dbus_get_changes_exception_by_version(calendar_h handle, const char *vie
                        ret = CALENDAR_ERROR_IPC;
                g_error_free(error);
                g_variant_unref(arg_handle);
-               g_variant_unref(arg_list);
+               if (arg_list != NULL)
+                       g_variant_unref(arg_list);
                return ret;
                /* LCOV_EXCL_STOP */
        }
@@ -967,7 +973,8 @@ int cal_dbus_insert_vcalendars(calendar_h handle, const char *stream,
                        ret = CALENDAR_ERROR_IPC;
                g_error_free(error);
                g_variant_unref(arg_handle);
-               g_variant_unref(arg_ids);
+               if (arg_ids != NULL)
+                       g_variant_unref(arg_ids);
                return ret;
                /* LCOV_EXCL_STOP */
        }