diff --git a/android/avrcp-lib.c b/android/avrcp-lib.c
index b7d0f5e..f3cdab9 100644
--- a/android/avrcp-lib.c
+++ b/android/avrcp-lib.c
&iov, 1);
}
+static int avrcp_status_rsp(struct avrcp *session, uint8_t transaction,
+ uint8_t pdu_id, uint8_t status)
+{
+ struct iovec iov;
+
+ if (status > AVRCP_STATUS_ADDRESSED_PLAYER_CHANGED)
+ return -EINVAL;
+
+ iov.iov_base = &status;
+ iov.iov_len = sizeof(status);
+
+ return avrcp_send_browsing(session, transaction, pdu_id, &iov, 1);
+}
+
int avrcp_set_browsed_player_rsp(struct avrcp *session, uint8_t transaction,
uint8_t status, uint16_t counter,
uint32_t items, uint8_t depth,
uint16_t len[UINT8_MAX];
int i;
- if (status != AVRCP_STATUS_SUCCESS) {
- iov[0].iov_base = &status;
- iov[0].iov_len = sizeof(status);
- return avrcp_send_browsing(session, transaction,
- AVRCP_SET_BROWSED_PLAYER,
- iov, 1);
- }
+ if (status != AVRCP_STATUS_SUCCESS)
+ return avrcp_status_rsp(session, transaction,
+ AVRCP_SET_BROWSED_PLAYER, status);
rsp.status = status;
put_be16(counter, &rsp.counter);
}
int avrcp_get_folder_items_rsp(struct avrcp *session, uint8_t transaction,
- uint16_t counter, uint8_t number,
- uint8_t *type, uint16_t *len,
- uint8_t **params)
+ uint8_t status, uint16_t counter,
+ uint8_t number, uint8_t *type,
+ uint16_t *len, uint8_t **params)
{
struct iovec iov[UINT8_MAX * 2 + 1];
struct get_folder_items_rsp rsp;
uint8_t item[UINT8_MAX][3];
int i;
- rsp.status = AVRCP_STATUS_SUCCESS;
+ if (status != AVRCP_STATUS_SUCCESS)
+ return avrcp_status_rsp(session, transaction,
+ AVRCP_GET_FOLDER_ITEMS, status);
+
+ rsp.status = status;
put_be16(counter, &rsp.counter);
put_be16(number, &rsp.number);
}
int avrcp_change_path_rsp(struct avrcp *session, uint8_t transaction,
- uint32_t items)
+ uint8_t status, uint32_t items)
{
struct iovec iov;
struct change_path_rsp rsp;
- rsp.status = AVRCP_STATUS_SUCCESS;
+ if (status != AVRCP_STATUS_SUCCESS)
+ return avrcp_status_rsp(session, transaction, AVRCP_CHANGE_PATH,
+ status);
+
+ rsp.status = status;
put_be32(items, &rsp.items);
iov.iov_base = &rsp;
if (number > AVRCP_MEDIA_ATTRIBUTE_LAST)
return -EINVAL;
- if (status != AVRCP_STATUS_SUCCESS) {
- iov[0].iov_base = &status;
- iov[0].iov_len = sizeof(status);
- return avrcp_send_browsing(session, transaction,
- AVRCP_GET_ITEM_ATTRIBUTES, iov, 1);
- }
+ if (status != AVRCP_STATUS_SUCCESS)
+ return avrcp_status_rsp(session, transaction,
+ AVRCP_GET_ITEM_ATTRIBUTES, status);
rsp.status = status;
rsp.number = number;
number * 2 + 1);
}
-int avrcp_play_item_rsp(struct avrcp *session, uint8_t transaction)
+int avrcp_play_item_rsp(struct avrcp *session, uint8_t transaction,
+ uint8_t status)
{
- struct iovec iov;
- struct play_item_rsp rsp;
-
- rsp.status = AVRCP_STATUS_SUCCESS;
-
- iov.iov_base = &rsp;
- iov.iov_len = sizeof(rsp);
-
- return avrcp_send_browsing(session, transaction, AVRCP_PLAY_ITEM,
- &iov, 1);
+ return avrcp_status_rsp(session, transaction, AVRCP_PLAY_ITEM,
+ status);
}
-int avrcp_search_rsp(struct avrcp *session, uint8_t transaction,
+int avrcp_search_rsp(struct avrcp *session, uint8_t transaction, uint8_t status,
uint16_t counter, uint32_t items)
{
struct iovec iov;
struct search_rsp rsp;
- rsp.status = AVRCP_STATUS_SUCCESS;
+ if (status != AVRCP_STATUS_SUCCESS)
+ return avrcp_status_rsp(session, transaction, AVRCP_SEARCH,
+ status);
+
+ rsp.status = status;
put_be16(counter, &rsp.counter);
put_be32(items, &rsp.items);
&iov, 1);
}
-int avrcp_add_to_now_playing_rsp(struct avrcp *session, uint8_t transaction)
+int avrcp_add_to_now_playing_rsp(struct avrcp *session, uint8_t transaction,
+ uint8_t status)
{
- struct iovec iov;
- struct add_to_now_playing_rsp rsp;
-
- rsp.status = AVRCP_STATUS_SUCCESS;
-
- iov.iov_base = &rsp;
- iov.iov_len = sizeof(rsp);
-
- return avrcp_send_browsing(session, transaction,
- AVRCP_ADD_TO_NOW_PLAYING, &iov, 1);
+ return avrcp_status_rsp(session, transaction, AVRCP_ADD_TO_NOW_PLAYING,
+ status);
}
int avrcp_send_passthrough(struct avrcp *session, uint32_t vendor, uint8_t op)
diff --git a/android/avrcp-lib.h b/android/avrcp-lib.h
index 9985c4a..2299c83 100644
--- a/android/avrcp-lib.h
+++ b/android/avrcp-lib.h
uint32_t items, uint8_t depth,
const char **folders);
int avrcp_get_folder_items_rsp(struct avrcp *session, uint8_t transaction,
- uint16_t counter, uint8_t number,
- uint8_t *type, uint16_t *len,
- uint8_t **params);
+ uint8_t status, uint16_t counter,
+ uint8_t number, uint8_t *type,
+ uint16_t *len, uint8_t **params);
int avrcp_change_path_rsp(struct avrcp *session, uint8_t transaction,
- uint32_t items);
+ uint8_t status, uint32_t items);
int avrcp_get_item_attributes_rsp(struct avrcp *session, uint8_t transaction,
uint8_t status, uint8_t number,
uint32_t *attrs, const char **text);
-int avrcp_play_item_rsp(struct avrcp *session, uint8_t transaction);
-int avrcp_search_rsp(struct avrcp *session, uint8_t transaction,
+int avrcp_play_item_rsp(struct avrcp *session, uint8_t transaction,
+ uint8_t status);
+int avrcp_search_rsp(struct avrcp *session, uint8_t transaction, uint8_t status,
uint16_t counter, uint32_t items);
-int avrcp_add_to_now_playing_rsp(struct avrcp *session, uint8_t transaction);
+int avrcp_add_to_now_playing_rsp(struct avrcp *session, uint8_t transaction,
+ uint8_t status);
int avrcp_send_passthrough(struct avrcp *session, uint32_t vendor, uint8_t op);
diff --git a/unit/test-avrcp.c b/unit/test-avrcp.c
index 154175e..a6a1872 100644
--- a/unit/test-avrcp.c
+++ b/unit/test-avrcp.c
if (start > 1)
return -ERANGE;
- avrcp_get_folder_items_rsp(session, transaction, 0xabcd, 0, NULL, NULL,
- NULL);
+ avrcp_get_folder_items_rsp(session, transaction, AVRCP_STATUS_SUCCESS,
+ 0xabcd, 0, NULL, NULL, NULL);
return -EAGAIN;
}
if (!uid)
return -ENOTDIR;
- avrcp_change_path_rsp(session, transaction, 0);
+ avrcp_change_path_rsp(session, transaction, AVRCP_STATUS_SUCCESS, 0);
return -EAGAIN;
}
if (!uid)
return -ENOENT;
- avrcp_play_item_rsp(session, transaction);
+ avrcp_play_item_rsp(session, transaction, AVRCP_STATUS_SUCCESS);
return -EAGAIN;
}
{
DBG("");
- avrcp_search_rsp(session, transaction, 0xaabb, 0);
+ avrcp_search_rsp(session, transaction, AVRCP_STATUS_SUCCESS, 0xaabb, 0);
return -EAGAIN;
}
if (!uid)
return -ENOENT;
- avrcp_add_to_now_playing_rsp(session, transaction);
+ avrcp_add_to_now_playing_rsp(session, transaction,
+ AVRCP_STATUS_SUCCESS);
return -EAGAIN;
}