Fix static analysis issues 61/297161/2
authorInkyun Kil <inkyun.kil@samsung.com>
Fri, 11 Aug 2023 05:39:36 +0000 (14:39 +0900)
committerInkyun Kil <inkyun.kil@samsung.com>
Fri, 11 Aug 2023 05:53:39 +0000 (14:53 +0900)
Change-Id: Ibb5f80f82a2da0633e02a8cb158375b2c8239d5a
Signed-off-by: Inkyun Kil <inkyun.kil@samsung.com>
src/eventsystem.c

index 553b29da7e15ee2611955539b0ee0ef7760e8853..e1ea13dfc1389bcbe6cfba63c02ce08ca2eb9ed1 100644 (file)
@@ -397,7 +397,7 @@ static int __get_gdbus_shared_connection(GDBusConnection **connection, GBusType
        if (*connection == NULL) {
                if (error != NULL) {
                        _E("Failed to get dbus [%s], bus_type [%d]",
-                               error->message, bus_type);
+                               error ? error->message : "", bus_type);
                        g_error_free(error);
                }
                return ES_R_ERROR;
@@ -431,7 +431,7 @@ static int __get_gdbus_shared_connection(GDBusConnection **connection, GBusType
                        if (conn_system == NULL) {
                                _E("failed to get system connection");
                                if (error != NULL) {
-                                       _E("error(%s)", error->message);
+                                       _E("error(%s)", error ? error ? error->message : "" : "");
                                        g_error_free(error);
                                }
                                return ES_R_ERROR;
@@ -842,7 +842,7 @@ static int __eventsystem_send_event(GDBusConnection *conn, eventinfo_s *evti, bu
        bundle_free_encoded_rawdata(&raw);
 
        if (ret == FALSE) {
-               _E("Unable to connect to dbus: %s", error->message);
+               _E("Unable to connect to dbus: %s", error ? error->message : "");
                g_error_free(error);
                return ES_R_ERROR;
        }
@@ -1147,7 +1147,7 @@ API int eventsystem_request_sending_system_event(const char *event_name,
                ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME,
                NULL, &error);
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1207,7 +1207,7 @@ static int __eventsystem_check_sender_validation(int sender_pid, const char *eve
                ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME,
                NULL, &error);
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1263,7 +1263,7 @@ static int __eventsystem_check_user_send_validation(const char *event_name)
                ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME,
                NULL, &error);
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1319,7 +1319,7 @@ static int __eventsystem_check_privilege_validation(const char *event_name)
                ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME,
                NULL, &error);
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1375,7 +1375,7 @@ static int __eventsystem_setup_trusted_peer(const char *event_name, const char *
                ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME,
                NULL, &error);
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1434,7 +1434,7 @@ static int __eventsystem_request_destination_list(const char *event_name, GList
                ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME,
                NULL, &error);
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1509,7 +1509,7 @@ static int __eventsystem_request_earlier_data(const char *event_name,
                NULL, &error);
 
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1573,7 +1573,7 @@ static int __request_esd_for_last_data(const char *event_name, bool check)
                NULL, &error);
 
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1637,7 +1637,7 @@ static int __eventsystem_launch_on_event_for_userevent(const char *event_name,
                ESD_BUS_NAME, ESD_OBJECT_PATH, ESD_INTERFACE_NAME,
                NULL, &error);
        if (proxy == NULL) {
-               _E("failed to create new proxy, error(%s)", error->message);
+               _E("failed to create new proxy, error(%s)", error ? error->message : "");
                g_error_free(error);
                ret = ES_R_ERROR;
                goto out_1;
@@ -1759,7 +1759,7 @@ static void _send_last_user_event(const char *event_name,
                                evti->is_trusted ? item->trusted_param : item->param,
                                &error);
                if (ret == FALSE) {
-                       _E("Unable to connect to dbus: %s", error->message);
+                       _E("Unable to connect to dbus: %s", error ? error->message : "");
                        g_error_free(error);
                }
        }