Modify subscribe for MC_SUBSCRIPTION_TYPE_PLAYBACK_ABILITY 26/211426/1 accepted/tizen/unified/20190805.042912 submit/tizen/20190805.013641
authorjiyong.min <jiyong.min@samsung.com>
Mon, 5 Aug 2019 00:11:12 +0000 (09:11 +0900)
committerjiyong.min <jiyong.min@samsung.com>
Mon, 5 Aug 2019 01:19:54 +0000 (10:19 +0900)
 - After mc_client_subscribe is called for MC_SUBSCRIPTION_TYPE_PLABACK_ABILITY,
   mc_playback_ability_updated_cb is not invoked
  When mc_client_subscribe is called, wrong user_data was pased.

Change-Id: I1184b5ef76b996b631219d6ccbe9c4ae0f6ba960

src/media_controller_client.c

index 5f4c5600ecd755e0f2bcc594ff54719f12467afc..e9295da190d86dc0b3249f06c2392d4a1996c50d 100644 (file)
@@ -286,7 +286,7 @@ static void __client_playback_ability_cb(const char *interface_name, const char
        mc_retm_if(mc_client == NULL, "mc_client is NULL");
 
        mc_playback_ability_updated_cb callback = (mc_playback_ability_updated_cb)mc_client->updated_cb[MC_CLIENT_EVENT_PLAYBACK_ABILITY].callback;
-       mc_retm_if(callback == NULL, "metadata_cb is NULL");
+       mc_retm_if(callback == NULL, "playback_ability_cb is NULL");
        mc_retm_if(message == NULL, "message is NULL");
 
        mc_secure_debug("__client_playback_ability_cb(%s, %s, %s, %s, %p)", interface_name, signal_name, message, request_id, user_data);
@@ -763,7 +763,7 @@ int mc_client_subscribe(mc_client_h client, const mc_subscription_type_e subscri
 
        ret = mc_ipc_unregister_listener(&mc_client->listeners, MC_DBUS_UPDATE_INTERFACE, cb_event[subscription_type].signal_name);
 
-       if (subscription_type == MC_SUBSCRIPTION_TYPE_METADATA) {
+       if (subscription_type == MC_SUBSCRIPTION_TYPE_METADATA || subscription_type == MC_SUBSCRIPTION_TYPE_PLAYBACK_ABILITY) {
                ret = __mc_client_register_filter_listener(mc_client, &mc_client->updated_cb[subscription_type].filter_list, server_name, cb_event[subscription_type].signal_name,
                                        cb_event[subscription_type].cb_func,  (void *)(mc_client));
        } else {