Modify subscribe for MC_SUBSCRIPTION_TYPE_PLAYBACK_ABILITY 27/211427/1 accepted/tizen/5.0/unified/20190805.035933 submit/tizen_5.0/20190805.013658
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:25:40 +0000 (01:25 +0000)
 - 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 0229ab95a33007aeb44fd30ba9b00f31149fc944..eab5835395615a4b0e689191410d66a7086e38fb 100644 (file)
@@ -307,7 +307,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);
@@ -841,7 +841,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 {