Fix invalid pointer casting 89/230489/2
authorInHong Han <inhong1.han@samsung.com>
Fri, 10 Apr 2020 06:06:08 +0000 (15:06 +0900)
committerInHong Han <inhong1.han@samsung.com>
Fri, 10 Apr 2020 09:08:43 +0000 (09:08 +0000)
Change-Id: Ib6eec53220eb03bd4a561b62e6ab03f2cbb641d6

client/sticker_dbus.c

index c6aaaea..f8a4b72 100644 (file)
@@ -933,7 +933,7 @@ int sticker_dbus_get_all_sticker_info(GDBusConnection *gdbus_connection, const c
     ret = _send_sync_message(gdbus_connection, g_variant_new("(sii)", app_id, offset, count), &reply, "get_all_sticker_info");
     if (ret == STICKER_CLIENT_ERROR_NONE) {
         reply_body = g_dbus_message_get_body(reply);
-        g_variant_get(reply_body, "(a(i))", &(*id_iter));
+        g_variant_get(reply_body, "(a(i))", id_iter);
     }
 
     if (reply_body)
@@ -954,7 +954,7 @@ int sticker_dbus_get_sticker_info_by_appid(GDBusConnection *gdbus_connection, co
     ret = _send_sync_message(gdbus_connection, g_variant_new("(sii)", app_id, offset, count), &reply, "get_sticker_info_by_appid");
     if (ret == STICKER_CLIENT_ERROR_NONE) {
         reply_body = g_dbus_message_get_body(reply);
-        g_variant_get(reply_body, "(a(i))", &(*id_iter));
+        g_variant_get(reply_body, "(a(i))", id_iter);
     }
 
     if (reply_body)
@@ -975,7 +975,7 @@ int sticker_dbus_get_sticker_info_by_type(GDBusConnection *gdbus_connection, con
     ret = _send_sync_message(gdbus_connection, g_variant_new("(siii)", app_id, (int)type, offset, count), &reply, "get_sticker_info_by_type");
     if (ret == STICKER_CLIENT_ERROR_NONE) {
         reply_body = g_dbus_message_get_body(reply);
-        g_variant_get(reply_body, "(a(i))", &(*id_iter));
+        g_variant_get(reply_body, "(a(i))", id_iter);
     }
 
     if (reply_body)
@@ -996,7 +996,7 @@ int sticker_dbus_get_sticker_info_by_group(GDBusConnection *gdbus_connection, co
     ret = _send_sync_message(gdbus_connection, g_variant_new("(ssii)", app_id, group, offset, count), &reply, "get_sticker_info_by_group");
     if (ret == STICKER_CLIENT_ERROR_NONE) {
         reply_body = g_dbus_message_get_body(reply);
-        g_variant_get(reply_body, "(a(i))", &(*id_iter));
+        g_variant_get(reply_body, "(a(i))", id_iter);
     }
 
     if (reply_body)
@@ -1017,7 +1017,7 @@ int sticker_dbus_get_sticker_info_by_keyword(GDBusConnection *gdbus_connection,
     ret = _send_sync_message(gdbus_connection, g_variant_new("(ssii)", app_id, keyword, offset, count), &reply, "get_sticker_info_by_keyword");
     if (ret == STICKER_CLIENT_ERROR_NONE) {
         reply_body = g_dbus_message_get_body(reply);
-        g_variant_get(reply_body, "(a(i))", &(*id_iter));
+        g_variant_get(reply_body, "(a(i))", id_iter);
     }
 
     if (reply_body)
@@ -1038,7 +1038,7 @@ int sticker_dbus_get_sticker_info_by_display_type(GDBusConnection *gdbus_connect
     ret = _send_sync_message(gdbus_connection, g_variant_new("(siii)", app_id, (int)type, offset, count), &reply, "get_sticker_info_by_disp_type");
     if (ret == STICKER_CLIENT_ERROR_NONE) {
         reply_body = g_dbus_message_get_body(reply);
-        g_variant_get(reply_body, "(a(i))", &(*id_iter));
+        g_variant_get(reply_body, "(a(i))", id_iter);
     }
 
     if (reply_body)
@@ -1134,7 +1134,7 @@ int sticker_dbus_get_recent_sticker_list(GDBusConnection *gdbus_connection, int
     ret = _send_sync_message(gdbus_connection, g_variant_new("(i)", count), &reply, "get_recent_sticker_info");
     if (ret == STICKER_CLIENT_ERROR_NONE) {
         reply_body = g_dbus_message_get_body(reply);
-        g_variant_get(reply_body, "(a(i))", &(*id_iter));
+        g_variant_get(reply_body, "(a(i))", id_iter);
     }
 
     if (reply_body)