diff options
author | Luiz Augusto von Dentz <luiz.von.dentz@intel.com> | 2017-05-01 18:40:59 +0300 |
---|---|---|
committer | Luiz Augusto von Dentz <luiz.von.dentz@intel.com> | 2017-05-02 14:22:37 +0300 |
commit | 535913b8ef94aa5cfac04add6f54dd18f207fbdd (patch) | |
tree | be481b5b1d627b3467fe00c94ae12349e89adc4c /unit/test-gatt.c | |
parent | d99df1d384750b2d578db6fc4679ecb091613e0f (diff) | |
download | bluez-535913b8ef94aa5cfac04add6f54dd18f207fbdd.tar.gz |
unit/test-gatt: Add test for unknown request
This send an unknown command (0xff) to test that no response is sent
since the spec mandates the server to not respond to commands:
BLUETOOTH SPECIFICATION Version 5.0 | Vol 3, Part F page 2179
'If a server receives a command that it does not support, indicated
by the Command Flag of the PDU set to one, then the server shall
ignore the Command.'
Diffstat (limited to 'unit/test-gatt.c')
-rw-r--r-- | unit/test-gatt.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/unit/test-gatt.c b/unit/test-gatt.c index 371cb467b..e0d29c77b 100644 --- a/unit/test-gatt.c +++ b/unit/test-gatt.c @@ -90,6 +90,11 @@ struct context { .size = sizeof(data(args)), \ } +#define false_pdu() \ + { \ + .valid = false, \ + } + #define define_test(name, function, type, bt_uuid, db, \ test_step, args...) \ do { \ @@ -403,6 +408,13 @@ static gboolean send_pdu(gpointer user_data) if (pdu->valid && (pdu->size == 0)) { test_debug("(no action expected)", "GATT: "); context->pdu_offset++; + + /* Quit the context if we processed the last PDU */ + if (!context->data->pdu_list[context->pdu_offset].valid) { + context_quit(context); + return FALSE; + } + return send_pdu(context); } @@ -4467,5 +4479,11 @@ int main(int argc, char *argv[]) raw_pdu(0xbf, 0x00), raw_pdu(0x01, 0xbf, 0x00, 0x00, 0x06)); + define_test_server("/robustness/unkown-command", + test_server, service_db_1, NULL, + raw_pdu(0x03, 0x00, 0x02), + raw_pdu(0xff, 0x00), + raw_pdu()); + return tester_run(); } |