diff options
| author | Ravi kumar Veeramally <ravikumar.veeramally@linux.intel.com> | 2013-11-05 23:09:16 +0200 |
|---|---|---|
| committer | Johan Hedberg <johan.hedberg@intel.com> | 2013-11-06 10:26:36 +0200 |
| commit | 1aee093b3eae7d989937068533468e185b85c393 (patch) | |
| tree | 2f3ed1c7a463c2f9f7feb42a218f79627a63e278 /android/hal-hidhost.c | |
| parent | aac7c55133b36ea207406068cf310cbab5260f14 (diff) | |
| download | bluez-1aee093b3eae7d989937068533468e185b85c393.tar.gz | |
android/hid: Handle invalid parameters in HAL
Diffstat (limited to 'android/hal-hidhost.c')
| -rw-r--r-- | android/hal-hidhost.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/android/hal-hidhost.c b/android/hal-hidhost.c index fe68fe794..815902b55 100644 --- a/android/hal-hidhost.c +++ b/android/hal-hidhost.c @@ -204,9 +204,8 @@ static bt_status_t hh_get_protocol(bt_bdaddr_t *bd_addr, case BTHH_BOOT_MODE: cmd.mode = HAL_HID_BOOT_PROTOCOL; break; - case BTHH_UNSUPPORTED_MODE: - cmd.mode = HAL_HID_UNSUPPORTED_PROTOCOL; - break; + default: + return BT_STATUS_PARM_INVALID; } return hal_ipc_cmd(HAL_SERVICE_ID_HIDHOST, @@ -236,9 +235,8 @@ static bt_status_t hh_set_protocol(bt_bdaddr_t *bd_addr, case BTHH_BOOT_MODE: cmd.mode = HAL_HID_BOOT_PROTOCOL; break; - case BTHH_UNSUPPORTED_MODE: - cmd.mode = HAL_HID_UNSUPPORTED_PROTOCOL; - break; + default: + return BT_STATUS_PARM_INVALID; } return hal_ipc_cmd(HAL_SERVICE_ID_HIDHOST, @@ -275,6 +273,8 @@ static bt_status_t hh_get_report(bt_bdaddr_t *bd_addr, case BTHH_FEATURE_REPORT: cmd.type = HAL_HID_FEATURE_REPORT; break; + default: + return BT_STATUS_PARM_INVALID; } return hal_ipc_cmd(HAL_SERVICE_ID_HIDHOST, HAL_OP_HID_GET_REPORT, @@ -309,6 +309,8 @@ static bt_status_t hh_set_report(bt_bdaddr_t *bd_addr, case BTHH_FEATURE_REPORT: cmd.type = HAL_HID_FEATURE_REPORT; break; + default: + return BT_STATUS_PARM_INVALID; } return hal_ipc_cmd(HAL_SERVICE_ID_HIDHOST, HAL_OP_HID_SET_REPORT, |