Fix the coverity issue (Dereference after null check)

[platform/core/connectivity/bluetooth-frwk.git] / bt-api / bt-avrcp.c

diff --git a/bt-api/bt-avrcp.c b/bt-api/bt-avrcp.c
index 4dc7173..9c6ba9a 100644 (file)
--- a/bt-api/bt-avrcp.c
+++ b/bt-api/bt-avrcp.c
@@ -24,11 +24,10 @@
 #include "bt-common.h"
 #include "bt-request-sender.h"
 #include "bt-event-handler.h"
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
 #include "bt-dpm.h"
 #endif
 
-
 BT_EXPORT_API int bluetooth_media_player_init(media_cb_func_ptr callback_ptr,
                                                void *user_data)
 {
@@ -72,7 +71,7 @@ BT_EXPORT_API int bluetooth_media_player_change_property(
 
        BT_CHECK_ENABLED(return);
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
                BT_ERR("Not allow to use AVRCP profile");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
@@ -101,7 +100,7 @@ BT_EXPORT_API int bluetooth_media_player_set_properties(
        BT_CHECK_PARAMETER(setting, return);
        BT_CHECK_ENABLED(return);
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
                BT_ERR("Not allow to use AVRCP profile");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
@@ -130,7 +129,7 @@ BT_EXPORT_API int bluetooth_media_player_change_track(
        BT_CHECK_PARAMETER(metadata, return);
        BT_CHECK_ENABLED(return);
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
                BT_ERR("Not allow to use AVRCP profile");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
@@ -228,7 +227,7 @@ BT_EXPORT_API int bluetooth_media_control_connect(
                return BLUETOOTH_ERROR_PERMISSION_DEINED;
        }
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_ADDRESS, remote_address) == BT_DPM_RESTRICTED) {
                BT_ERR("Blacklist device");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
@@ -293,6 +292,80 @@ BT_EXPORT_API int bluetooth_media_control_disconnect(
        return result;
 }
 
+BT_EXPORT_API int bluetooth_media_target_connect(
+                       bluetooth_device_address_t *remote_address)
+{
+       int result;
+       bt_user_info_t *user_info;
+
+       BT_CHECK_PARAMETER(remote_address, return);
+       BT_CHECK_ENABLED(return);
+
+       if (_bt_check_privilege(BT_CHECK_PRIVILEGE, BT_AVRCP_TARGET_CONNECT)
+                       == BLUETOOTH_ERROR_PERMISSION_DEINED) {
+               BT_ERR("Don't have a privilege to use this API");
+               return BLUETOOTH_ERROR_PERMISSION_DEINED;
+       }
+
+#ifdef TIZEN_FEATURE_BT_DPM
+       if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
+               BT_ERR("Not allow to use AVRCP profile");
+               return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
+       }
+#endif
+       user_info = _bt_get_user_data(BT_AVRCP);
+       retv_if(user_info->cb == NULL, BLUETOOTH_ERROR_INTERNAL);
+
+       BT_INIT_PARAMS();
+       BT_ALLOC_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       g_array_append_vals(in_param1, remote_address,
+                                       sizeof(bluetooth_device_address_t));
+
+       result = _bt_send_request_async(BT_BLUEZ_SERVICE,
+                               BT_AVRCP_TARGET_CONNECT, in_param1,
+                               in_param2, in_param3, in_param4,
+                               user_info->cb, user_info->user_data);
+
+       BT_FREE_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       return result;
+}
+
+BT_EXPORT_API int bluetooth_media_target_disconnect(
+                       bluetooth_device_address_t *remote_address)
+{
+       int result;
+       bt_user_info_t *user_info;
+
+       BT_CHECK_PARAMETER(remote_address, return);
+       BT_CHECK_ENABLED(return);
+
+       if (_bt_check_privilege(BT_CHECK_PRIVILEGE, BT_AVRCP_TARGET_DISCONNECT)
+                               == BLUETOOTH_ERROR_PERMISSION_DEINED) {
+               BT_ERR("Don't have a privilege to use this API");
+               return BLUETOOTH_ERROR_PERMISSION_DEINED;
+       }
+
+       user_info = _bt_get_user_data(BT_AVRCP);
+       retv_if(user_info->cb == NULL, BLUETOOTH_ERROR_INTERNAL);
+
+       BT_INIT_PARAMS();
+       BT_ALLOC_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       g_array_append_vals(in_param1, remote_address,
+                                       sizeof(bluetooth_device_address_t));
+
+       result = _bt_send_request_async(BT_BLUEZ_SERVICE,
+                               BT_AVRCP_TARGET_DISCONNECT, in_param1,
+                               in_param2, in_param3, in_param4,
+                               user_info->cb, user_info->user_data);
+
+       BT_FREE_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       return result;
+}
+
 BT_EXPORT_API int bluetooth_media_control_command(
                                                media_player_control_cmd type)
 {
@@ -300,13 +373,12 @@ BT_EXPORT_API int bluetooth_media_control_command(
 
        BT_CHECK_ENABLED(return);
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
                BT_ERR("Not allow to use AVRCP profile");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
        }
 #endif
-
        BT_INIT_PARAMS();
        BT_ALLOC_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
 
@@ -320,6 +392,35 @@ BT_EXPORT_API int bluetooth_media_control_command(
        return result;
 }
 
+BT_EXPORT_API int bluetooth_media_control_command_to_dest(
+                                               media_player_control_cmd type,
+                                               bluetooth_device_address_t *remote_address)
+{
+       int result;
+
+       BT_CHECK_ENABLED(return);
+
+#ifdef TIZEN_FEATURE_BT_DPM
+       if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
+               BT_ERR("Not allow to use AVRCP profile");
+               return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
+       }
+#endif
+       BT_INIT_PARAMS();
+       BT_ALLOC_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       g_array_append_vals(in_param1, &type, sizeof(int));
+       g_array_append_vals(in_param2, remote_address,
+                                               sizeof(bluetooth_device_address_t));
+
+       result = _bt_send_request(BT_BLUEZ_SERVICE, BT_AVRCP_HANDLE_CONTROL_TO_DEST,
+               in_param1, in_param2, in_param3, in_param4, &out_param);
+
+       BT_FREE_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       return result;
+}
+
 BT_EXPORT_API int bluetooth_media_control_set_property(
                                                media_player_property_type type,
                                                unsigned int value)
@@ -328,7 +429,7 @@ BT_EXPORT_API int bluetooth_media_control_set_property(
 
        BT_CHECK_ENABLED(return);
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
                BT_ERR("Not allow to use AVRCP profile");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
@@ -360,7 +461,7 @@ BT_EXPORT_API int bluetooth_media_control_get_property(
        BT_CHECK_PARAMETER(value, return);
        BT_CHECK_ENABLED(return);
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
                BT_ERR("Not allow to use AVRCP profile");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
@@ -384,16 +485,47 @@ BT_EXPORT_API int bluetooth_media_control_get_property(
        return result;
 }
 
+BT_EXPORT_API int bluetooth_media_transport_set_property(
+                                               media_transport_property_type type,
+                                               unsigned int value)
+{
+       int result;
+
+       BT_CHECK_ENABLED(return);
+
+#ifdef TIZEN_FEATURE_BT_DPM
+       if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
+               BT_ERR("Not allow to use AVRCP profile");
+               return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
+       }
+#endif
+
+       BT_INIT_PARAMS();
+       BT_ALLOC_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       g_array_append_vals(in_param1, &type, sizeof(int));
+       g_array_append_vals(in_param2, &value, sizeof(unsigned int));
+
+       result = _bt_send_request(BT_BLUEZ_SERVICE,
+                               BT_AVRCP_TRANSPORT_SET_PROPERTY,
+                               in_param1, in_param2, in_param3,
+                               in_param4, &out_param);
+
+       BT_FREE_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);
+
+       return result;
+}
+
 BT_EXPORT_API int bluetooth_media_control_get_track_info(
                media_metadata_attributes_t *metadata)
 {
        int result;
-       media_metadata_t meta_data;
+       media_metadata_t *meta_data;
 
        BT_CHECK_PARAMETER(metadata, return);
        BT_CHECK_ENABLED(return);
 
-#ifdef TIZEN_DPM_ENABLE
+#ifdef TIZEN_FEATURE_BT_DPM
        if (_bt_check_dpm(BT_DPM_AVRCP, NULL) == BT_DPM_RESTRICTED) {
                BT_ERR("Not allow to use AVRCP profile");
                return BLUETOOTH_ERROR_DEVICE_POLICY_RESTRICTION;
@@ -406,17 +538,15 @@ BT_EXPORT_API int bluetooth_media_control_get_track_info(
        result = _bt_send_request(BT_BLUEZ_SERVICE, BT_AVRCP_GET_TRACK_INFO,
                in_param1, in_param2, in_param3, in_param4, &out_param);
 
-       memset(&meta_data, 0x00, sizeof(media_metadata_t));
-
-       meta_data = g_array_index(out_param, media_metadata_t, 0);
+       meta_data = &g_array_index(out_param, media_metadata_t, 0);
 
-       metadata->title = g_strdup(meta_data.title);
-       metadata->artist = g_strdup(meta_data.artist);
-       metadata->album = g_strdup(meta_data.album);
-       metadata->genre = g_strdup(meta_data.genre);
-       metadata->total_tracks = meta_data.total_tracks;
-       metadata->number = meta_data.number;
-       metadata->duration = (int64_t) meta_data.duration;
+       metadata->title = g_strdup(meta_data->title);
+       metadata->artist = g_strdup(meta_data->artist);
+       metadata->album = g_strdup(meta_data->album);
+       metadata->genre = g_strdup(meta_data->genre);
+       metadata->total_tracks = meta_data->total_tracks;
+       metadata->number = meta_data->number;
+       metadata->duration = (int64_t) meta_data->duration;
 
        BT_FREE_PARAMS(in_param1, in_param2, in_param3, in_param4, out_param);