| d31dfe8c |
mgmt-tester: Enable Kernel Debug Experimental Feature
This attempts to enable Kernel Debug Experimental Feature if debug is
enabled. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +27, -0 |
| 1285bca9 |
Release 5.59 |
Marcel Holtmann |
4 years ago |
2 files, +7, -1 |
| de948fc8 |
Revert "doc/mgmt-api - Add a new error code for HCI status 0x3e"
This reverts commit 9ad67264d97b882fa58391982ecb5b003368fef0. |
Marcel Holtmann |
4 years ago |
1 file, +0, -2 |
| 11a00179 |
mgmt-tester: Fix adding padding for Ext Adv (Scan Response) Data
These commands are actually of variable size so they don't expect
padding like in the non-extended version. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +3, -51 |
| f465a00f |
btdev: Check advertising/scanning states when changing Resolving List
Resolving List cannot be changed when advertising/scanning:
• Advertising (other than periodic advertising) is enabled,
• Scanning is enabled, or
• an HCI_LE_Create_Connection, HCI_LE_Extended_Create_Connection, or
HCI_LE_Periodic_Advertising_Create_Sync command is outstanding. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +40, -0 |
| 6031e6b8 |
btdev: Check advertising/scanning states when changing White List
White List cannot be changed when advertising/scanning:
• any advertising filter policy uses the White List and advertising is
enabled,
• the scanning filter policy uses the White List and scanning is
enabled, or
• the initiator filter policy uses the White List and an
HCI_LE_Create_Connection or HCI_LE_Extended_Create_Connection
command is outstanding. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +68, -0 |
| 125fa19f |
btdev: Remove debugs
Remove left-over util_debugs. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +0, -9 |
| 2ab15d89 |
tools/mgmt-tester: Fix expected HCI command accounting
This fixes test condition count in the expected HCI command
callback.
When the expected HCI opcode is detected, mark the condition
as done. Any subsequent HCI commands are ignored.
Without this fix, in couple of test cases where the expected HCI
command is detected more than once, the test may be erroneously
reported as a failure or prematurely declared as a success before
waiting on an expected MGMT event condition.
The test cases where this behavior is fixed:
Remove Ext Advertising - Success 1
Remove Ext Advertising - Success 2
Reviewed-by: Tedd Ho-Jeong An <tedd.an@intel.com> |
Inga Stotland |
4 years ago |
1 file, +4, -1 |
| 6efdbd8d |
shared/mgmt: Fix not processing request queue
If the kernel returns an invalid opcode like below the request won't
be processed making it unusable:
@ MGMT Command: Read Advertising Features (0x003d) plen 0
{0x0001} [hci0] 14:25:11.096370
@ MGMT Event: Command Status (0x0002) plen 3
{0x0001} [hci0] 14:25:11.096373
Set Advertising (0x0029)
Status: Not Supported (0x0c)
In order to fix this attempts to remove the first request pending on the
given index:
= bluetoothd: src/advertising.c:read_adv_features_callback() Failed to
read advertising features: Not Supported (0x0c) |
Luiz Augusto von Dentz |
4 years ago |
1 file, +11, -0 |
| f19000cf |
btdev: Fix handling of white/resolving list
le_cig.cis entries was not properly checked which could lead to
overwriting le_wl/le_rl field.
While at it this also makes changes so have proper struct/fields for
le_wl and le_rl so they easier to manage. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +103, -55 |
| e6310300 |
btdev: Fix crash when generating disconnected event for BIS
When generating disconnected event for a BIS it would not have any
link as the ISO is in fact we are broadcasting and don't have an ACL. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +4, -2 |
| 4e6e2a10 |
tools/mgmt-tester: Add setup condition to resolve list
This adds expected setup HCI command condition to
"Start Discovery - Disable Resolve List" case.
Check that on setup HCI command Set LE Resolve Enable is complete
with "enable" parameter set to 1. |
Inga Stotland |
4 years ago |
1 file, +10, -0 |
| 971c46f0 |
tools/mgmt-tester: Adjust test setup complete check
This changes check for setup complete in client_cmd_complete() callback
from tester_setup_complete() to test_setup_condition_complete(). This
allows for combining setup conditions when setup_bthost() is called.
Reviewed-by: Tedd Ho-Jeong An <tedd.an@intel.com> |
Inga Stotland |
4 years ago |
1 file, +2, -1 |
| fa0915d8 |
tools/mgmt-tester: Add tests for whitelist and resolving list
This patch adds test cases for checking the whitelist and resolving list
HCI commands when the LL Privacy feature is enabled. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +363, -0 |
| 5e162f0b |
tools/mgmt-tester: Add tests for experimental feature command
This patch adds test cases for Read/Set Experimenal Feature commands. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +137, -0 |
| 76f49e69 |
emulator/btdev: Enable LE Privacy feature
This patch enables LE Privacy to support LL Privacy feature. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +1, -0 |
| d7c2a4ce |
tools/mgmt-tester: Add test cases for get/set device flags
This patch adds test cases for get/set device flags commands. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +182, -0 |
| 616dd86b |
tools/mgmt-tester: Print hexdump if mgmt event parameters not match
This patch displays the hexdump of expected and received mgmt event
parameters. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +3, -0 |
| 9f5e0710 |
lib: Align mgmt header with document
This patch updates the mgmt header to align with the mgmt api document
(doc/mgmt-api.txt) |
Tedd Ho-Jeong An |
4 years ago |
1 file, +6, -1 |
| d70618e4 |
tools/btmgmt: Add device flags commands
This patch adds commands for get/set device flags mgmt ops.
Usage:
get-flags [-t type] <address>
-t type Address Type
0 BR/EDR
1 LE Public
2 LE Random
set-flags [-f flags] [-t type] <address>
-t type Address Type
0 BR/EDR
1 LE Public
2 LE Random
-f flags Device flag.
1 Remote Wake Enable |
Tedd Ho-Jeong An |
4 years ago |
1 file, +174, -0 |
| 88986355 |
mgmt: Add missing opcodes to mgmt_op
This adds the missing opcodes to mgmt_op so they are properly decoded. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +3, -0 |
| 6052a8e0 |
lib/uuid: Fix string to uuid32 conversion
Use strtoul to prevent 32 bit overflow
Reviewed-by: Yun-Hao Chung <howardchung@chromium.org> |
Archie Pusaka |
4 years ago |
1 file, +1, -1 |
| 2501575a |
core: Add RSSI sampling period in system parameter
Add an option in main.conf to configure the default RSSI sampling
period for advertisement monitor.
Reviewed-by: apusaka@chromium.org
Reviewed-by: mmandlik@chromium.org
Reviewed-by: mcchou@chromium.org |
Yun-Hao Chung |
4 years ago |
4 files, +34, -2 |
| af77ecf5 |
tools/mgmt-tester: Fix "Remove Ext Advertising" case
This fixes the expected behavior of the following test case:
"Remove Ext Advertising - Success 1", when the fail status
was erroneously overwritten with success:
<log snip>
New Advertising Removed event received
Test condition complete, 2 left
HCI Command 0x2039 length 6
Invalid parameter size for HCI command
Remove Ext Advertising - Success 1 - test failed
HCI Command 0x203c length 1
HCI Command 0x2039 length 2
Test condition complete, 1 left
Remove Advertising (0x003f): Success (0x00)
Test condition complete, 0 left
Remove Ext Advertising - Success 1 - test passed
The expected HCI command to disable extended advertisement for a
non-zero number of sets should contain the specified number the
advertising sets.
In this particular test case, number of sets is set to 1 and,
as a result, the expected HCI command should be:
"00 | 01 | 01 00 00 00"
disable | number of sets | set info
Also, to avoid false positives/negatives, skip exmination of HCI
commands after the test conditions are met. |
Inga Stotland |
4 years ago |
1 file, +7, -3 |
| df6e6b73 |
avrcp: Fix unregister AVRCP player
'notify_addressed_player_changed()' expected to be called with
'player->changed_id' set to a non-zero value.
player->changed_id = g_idle_add(notify_addressed_player_changed,
player);
And 'avrcp_player_event()' relies on 'player->changed_id' to perform
Addressed Player Changed notification. However,
'avrcp_unregister_player()' calls 'notify_addressed_player_changed()'
without adding it to the main loop and set 'player->changed_id'. To
indicate addreddsed player changed for both scenarios, we set
'player->changed_id' to 1 at the head of
'notify_addressed_player_changed()'.
Fixes https://github.com/bluez/bluez/issues/142 |
Huang-Huang Bao |
4 years ago |
1 file, +6, -0 |
| 9e56bced |
src/main: Fix double free str
This patch fixes the double free the string.
The string variable is always freed within its if statement so no need
to free it again in the end. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +0, -1 |
| c45fd229 |
tools/mgmt-tester: Add test cases for remove uuid command
This patch adds test cases for the Remove UUID command. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +143, -0 |
| bc1e7f84 |
tools/mgmt-tester: Add test cases for set device id command
This patch adds test cases for the Set Device ID command. |
Tedd Ho-Jeong An |
4 years ago |
1 file, +163, -0 |
| f7a568d7 |
monitor: Decode MSFT LE Monitor Advertisement |
Marcel Holtmann |
4 years ago |
1 file, +28, -1 |
| 2058aadd |
mesh: Fix race condition memory leak
This is a minor fix of a memory leak triggered on process exit if
proceess has been killed right after requesting an outbound
advertisement be sent. It is harmless, but will cause an occasional
static analysis failure. |
Brian Gix |
4 years ago |
1 file, +2, -0 |
| 9260107b |
mesh: Add single threading to prov-acp ob messaging
Certain IOP and stress testing scenarios can cause additional outbound
messages to be attempted before the prior outbound message completes.
This patch adds queuing if outbpund messages are attempted too quickly
so that the additional message is not sent until the prior message is
ACKed. |
Brian Gix |
4 years ago |
2 files, +56, -33 |
| 7f0a643d |
test/mesh: Add support for testing more OOB auth
To pass IOP testing, we need to be able to support Input OOB and Out of
band Public Key exchange, This patch adds agent methods to support
those capabilities, but are turned off because the test code uses
insecure fixed values instead of randomized values. |
Brian Gix |
4 years ago |
1 file, +23, -0 |
| 1441c240 |
tools/mesh: Add all supported OOB methods to cfgclient
To support the widest range of mesh devices, we need to support any
possible capability combinations that a remote device may request. |
Brian Gix |
4 years ago |
1 file, +27, -5 |
| 9fd05a6e |
mesh: Normalize endian of public/private ECC keys
The Mesh profile specification defines a Mesh byte order of Big Endian
for Public keys used to calculate shared secrets. Further the
specification sample data also show this same byte order for Private
keys. However, our internal ECDH shared secret calculation requires
Little Endian byte ordering. This fixes our DBus interface, and
debugging output to use Mesh Byte Ordering (Big Endian) for all human
readable input/output. |
Brian Gix |
4 years ago |
2 files, +10, -1 |
| 01404f57 |
mesh: Fix delivery of PB-ACK to acceptors
Remove unneeded initialization that prevented the Acceptor roll of ever
recognizing a valid PB-ACK. |
Brian Gix |
4 years ago |
1 file, +0, -1 |
| 980bd585 |
mesh: Add manpage for bluetooth-meshd daemon
This adds manpage with a brief description of BLuettoth Mesh daemon
command line options. |
Inga Stotland |
4 years ago |
4 files, +83, -1 |
| 47f51a98 |
core: Fix loading AVDTP options
Fix misassigned AVDTP StreamMode option, which causes StreamMode not
being set.
Fix AVDTP SessionMode option not freed.
Reviewed-by: mmandlik@chromium.org
Reviewed-by: apusaka@chromium.org
Reviewed-by: Tedd Ho-Jeong An <tedd.an@intel.com> |
Yun-Hao Chung |
4 years ago |
1 file, +3, -1 |
| 8b51fbcb |
Fix memory leaks in obex
This patch fixes a couple memory leaks.
In filesystem, g_file_get_contents allocates fresh memory to buf.
It needs to be freed after conversion to a GString object.
Destination was missed on an error path as is mld. |
Steve Grubb |
4 years ago |
2 files, +7, -2 |
| c66ef8ca |
core: Enable experimental MGMT when experimental is set
This enables experimental MGMT features when experimental has been set. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +116, -10 |
| 52de2520 |
btmon: Use bt_uuid128_to_str when printing UUIDs of 128 bits
This ensures that know UUIDs names are printed including those used by
experimental features. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +6, -21 |
| afe41d26 |
shared/util: Decode BlueZ experimental UUIDs
This adds BlueZ experimental UUIDs to uuid128_table so they are
decoded by the likes of btmon. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +6, -0 |
| 9ea7a92c |
shared/util: Add bt_uuid128_to_str
This adds bt_uuid128_to_str which can be used to convert UUID 128 bit
binary format into string. |
Luiz Augusto von Dentz |
4 years ago |
2 files, +13, -0 |
| 6a922904 |
btdev: Add resolving list support
This adds support for commands related to resolving list. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +210, -1 |
| 78902b77 |
btdev: Add support for Add/Remove White List
This adds support for Add to/Remove from White List. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +100, -4 |
| c3159848 |
client: Fix advertise broadcast
When advertising with type broadcast it cannot be discoverable as that
would require setting flags which is not allowed by the spec. |
Luiz Augusto von Dentz |
4 years ago |
1 file, +3, -0 |
| 5dba6908 |
main.conf: Add option to enable experimental features
The adds options Experimental which is the equivalent to
-E/--experimental command line option.
Fixes https://github.com/bluez/bluez/issues/132 |
Luiz Augusto von Dentz |
4 years ago |
3 files, +15, -3 |
| 31ce1e9a |
tools: Fix memory leaks
req is allocated memory that must be freed. It appears all other error
paths got fail which handles this.
g_new0 allocates memory to data which must be freed.
g_malloc0 allocates memory to comp which must be freed. |
Steve Grubb |
4 years ago |
3 files, +11, -4 |
| ca769001 |
core: Fix memory leaks
g_key_file_get_string allocates memory that must be freed.
asprintf allocates memory that must be freed. |
Steve Grubb |
4 years ago |
2 files, +2, -0 |
| 8d61942f |
profile: Fix memory leaks
g_new0 allocates memory that must be freed |
Steve Grubb |
4 years ago |
1 file, +3, -1 |
| 28ce42b0 |
plugin: Fix memory leaks
g_file_get_contents allocates memory. It needs to be freed on any
function exit. |
Steve Grubb |
4 years ago |
1 file, +1, -2 |