diff options
| author | Jakub Tyszkowski <jakub.tyszkowski@tieto.com> | 2014-01-22 09:25:18 +0100 |
|---|---|---|
| committer | Szymon Janc <szymon.janc@gmail.com> | 2014-01-23 23:42:30 +0100 |
| commit | 177fe93488cd75272f4afe168b0e60ac04ba8585 (patch) | |
| tree | 73dacda344f503d44fd4aa1ef00f33ae9b0a5b9a /android/ipc-tester.c | |
| parent | fddce77253c20f4d97e45a8fd5b6620f74f45742 (diff) | |
| download | bluez-177fe93488cd75272f4afe168b0e60ac04ba8585.tar.gz | |
android/ipc-tester: Add case for HIDHOST Set Info
This patch adds test for verifying data length inside
hal_cmd_hidhost_set_info struct.
Diffstat (limited to 'android/ipc-tester.c')
| -rw-r--r-- | android/ipc-tester.c | 49 |
1 files changed, 49 insertions, 0 deletions
diff --git a/android/ipc-tester.c b/android/ipc-tester.c index 2871587a9..641bc5bb8 100644 --- a/android/ipc-tester.c +++ b/android/ipc-tester.c @@ -650,6 +650,41 @@ struct hal_hdr enable_bt_service_hdr = { .len = 0, }; +struct hidhost_set_info_data { + struct hal_hdr hdr; + struct hal_cmd_hidhost_set_info info; + + /* data placeholder for hal_cmd_hidhost_set_info.descr[0] field */ + uint8_t buf[BLUEZ_HAL_MTU - sizeof(struct hal_hdr) - + sizeof(struct hal_cmd_hidhost_set_info)]; +} __attribute__((packed)); + +#define set_info_data "some descriptor" + +static struct hidhost_set_info_data hidhost_set_info_data_overs = { + .hdr.service_id = HAL_SERVICE_ID_HIDHOST, + .hdr.opcode = HAL_OP_HIDHOST_SET_INFO, + .hdr.len = sizeof(struct hal_cmd_hidhost_set_info) + + sizeof(set_info_data), + + /* declare wrong descriptor length */ + .info.descr_len = sizeof(set_info_data) + 1, + /* init .info.descr[0] */ + .buf = set_info_data, +}; + +static struct hidhost_set_info_data hidhost_set_info_data_unders = { + .hdr.service_id = HAL_SERVICE_ID_HIDHOST, + .hdr.opcode = HAL_OP_HIDHOST_SET_INFO, + .hdr.len = sizeof(struct hal_cmd_hidhost_set_info) + + sizeof(set_info_data), + + /* declare wrong descriptor length */ + .info.descr_len = sizeof(set_info_data) - 1, + /* init .info.descr[0] */ + .buf = set_info_data, +}; + int main(int argc, char *argv[]) { snprintf(exec_dir, sizeof(exec_dir), "%s", dirname(argv[0])); @@ -915,6 +950,20 @@ int main(int argc, char *argv[]) HAL_OP_HIDHOST_SET_INFO, sizeof(struct hal_cmd_hidhost_set_info), -1, HAL_SERVICE_ID_BLUETOOTH, HAL_SERVICE_ID_HIDHOST); + test_generic("Data size HIDHOST Set Info Vardata+", + ipc_send_tc, setup, teardown, + &hidhost_set_info_data_overs, + (sizeof(struct hal_hdr) + + sizeof(struct hal_cmd_hidhost_set_info) + + sizeof(set_info_data)), + HAL_SERVICE_ID_BLUETOOTH, HAL_SERVICE_ID_HIDHOST); + test_generic("Data size HIDHOST Set Info Vardata-", + ipc_send_tc, setup, teardown, + &hidhost_set_info_data_unders, + (sizeof(struct hal_hdr) + + sizeof(struct hal_cmd_hidhost_set_info) + + sizeof(set_info_data)), + HAL_SERVICE_ID_BLUETOOTH, HAL_SERVICE_ID_HIDHOST); test_datasize_valid("HIDHOST Get Protocol+", HAL_SERVICE_ID_HIDHOST, HAL_OP_HIDHOST_GET_PROTOCOL, sizeof(struct hal_cmd_hidhost_get_protocol), 1, |