Commit Graph

304 Commits (master)

Author SHA1 Message Date
Mathieu Camélique 3746f5af68 change type in ScanParameters getters 2024-05-22 08:52:42 +02:00
Mathieu Camélique 17a03ab8e6 correct scan parameters types 2024-05-16 12:39:40 +02:00
Mingjie Shen 378f2f579d Fix null pointer dereferencing
Add null check for return values of functions that are mostly (but not
always) checked for null.
E.g., since 98% of calls to protocol_stack_interface_info_get_by_id
check for null, it is likely that the function can return null values in
some cases, and omitting the check could crash the program.
2023-04-20 00:54:47 -04:00
Vincent Coubard c34640495f
Bluetooth: Inform privacy risk of using signed writes.
The Cordio stack uses a single CSRK. It can be used by a
malicious device to track the Mbed OS application if signed
writes are used.

Signed-off-by: Vincent Coubard <vincent.coubard@arm.com>
2022-05-06 13:44:33 +01:00
Lingkai Dong db8852c5d8 CMake: connectivity: Guard unit test directories
When unit tests or unit test stubs get added as CMake targets, they
becomes part of the "all" target and get compiled when building the
whole project. When building greentea tests we need to disable unit
tests and stubs to avoid unnecessary compilation and errors.
2021-09-20 16:28:54 +01:00
Martin Kojtal 3318720ada
Merge pull request #14941 from hazzlim/update-tests-to-use-chrono
Update references to time values to use chrono
2021-07-30 12:02:10 +01:00
harmut01 86c2d70a5b Greentea: Remove deprecated APIs and use chrono
References to time should do so using std::chrono. We reworked tests in
connectivity and drivers to use std::chrono and new APIs in order to
remove deprecation warnings resulting from deprecated API calls.
This required addition of  a macro for test assertions using std::chrono
values.

As host test "timing_drift_auto" expects time values represented as an
integral number of microseconds, we explicitly provide this in place
using "microseconds{TICKER_TIME}.count()" in the relevant ticker tests.
We recognise this is ugly, but thought it best to descriptively convert
from std::chrono to the host test's required representation.

Co-authored-by: Hari Limaye <hari.limaye@arm.com>
2021-07-29 14:06:27 +01:00
Jaeden Amero 4480abeaf9 cmake: Avoid linking to gcov
Use the `--coverage` option instead of manually linking to gcov, as some
host platforms (like macOS, FreeBSD) don't have gcov by default and use
an llvm equivalent instead.
2021-07-22 15:41:49 +01:00
Rajkumar Kanagaraj fb5b0ede45 CMake: Limit the mbed-fakes-ble to required dependency mock 2021-07-21 01:56:29 -07:00
Rajkumar Kanagaraj 9df7a50f39 CMake: Move ble fakes into FEATURE_BLE double dir
Previously ble fakes as part of UNITTESTS/fakes, this PR moves
ble fakes to FEATURE_BLE double directory to make ble stubs
to be self-contained.
2021-07-21 01:56:29 -07:00
Rajkumar Kanagaraj a61392ae30 CMake: Create mbed-headers-ble library
Previously ble headers are part of mbed-headers-connectivity, this PR
moves ble headers into new mbed-headers-ble to make ble stubs to be
more self-contained and improves the composition of the library.
2021-07-21 01:51:48 -07:00
Martin Kojtal c9234177c1
Merge pull request #14921 from paul-szczepanek-arm/fix-ble-default-db
BLE: suppress error when no path is passed in for ble security db
2021-07-20 10:02:28 +02:00
Paul Szczepanek 8a4b278a38 suppress error when no path is passed in for ble sec db 2021-07-14 17:26:57 +01:00
Paul Szczepanek 7ba7f3e667 fix toggling off ble feature signing 2021-07-14 15:13:13 +01:00
Martin Kojtal 3c2c5be298
Merge pull request #14824 from paul-szczepanek-arm/persistence-fix
BLE: Manual BLE security manager db synchronisation
2021-07-12 13:51:50 +02:00
Martin Kojtal d83494f54b
Merge pull request #14820 from paul-szczepanek-arm/events-doc
BLE: Clarify ble event docs with crossreferences
2021-07-01 17:17:49 +02:00
Hari Limaye f95052cf6f CMake: Require TEST_SOURCES in greentea CMake file
Assumption that greentea test file is always named main.cpp is
incorrect. Updated mbed_greentea_add_test() macro to make TEST_SOURCES
parameter compulsory, which is used to specify greentea test
file(s). This allows tests to use C, or have a different name.
Therefore also updated all pre-existing greentea test CMake files to
explicity add main.cpp to TEST_SOURCES.
2021-06-29 13:33:42 +01:00
Paul Szczepanek 1ba5b25ad9 fix calls with params in docs so they are referenced correclty 2021-06-29 10:17:50 +01:00
Paul Szczepanek 470917f9ab add implicit call to preserve state on reset 2021-06-28 19:38:13 +01:00
Paul Szczepanek 25211f5750 missing cross refernces and grammar 2021-06-28 16:32:26 +01:00
Paul Szczepanek c9d0ff8674 ble security db complete sync implementation 2021-06-23 22:25:00 +01:00
Paul Szczepanek 7b4d1d59f1 add new BLE API call to sync security db with persisten storage 2021-06-23 22:25:00 +01:00
Paul Szczepanek 55657c588b clarify ble event docs with corssreferences 2021-06-23 13:27:15 +01:00
Paul Szczepanek ede39c4608 put statements inside the correct feature guard blocks 2021-06-17 16:51:28 +01:00
Martin Kojtal 8cb8e49a67
Merge pull request #14747 from paul-szczepanek-arm/fix-adv-con
BLE: fix advertising set termination event
2021-06-17 15:37:29 +02:00
Martin Kojtal df12718139
Merge pull request #14737 from paul-szczepanek-arm/cmake-ble-rebased
BLE: Add cmake unittest fakes for BLE and events
2021-06-11 14:04:47 +02:00
Paul Szczepanek afeb696d01 fix CORDIO not passing the conn id in param for adv stop event
the hci handle is different from host handle, this is how it's done in conn open event, we carry the conn id in the hdr.param
2021-06-09 18:50:17 +01:00
Paul Szczepanek 963657bfb0 translate hci conn handle to host handle for adv stop event 2021-06-09 18:50:17 +01:00
Paul Szczepanek 6b930d6449 translate adv set error codes and only act on successful stop 2021-06-09 18:50:17 +01:00
Paul Szczepanek 4fae828987 Fix process enable queue guard not being reset on failed runs 2021-06-09 18:50:17 +01:00
Paul Szczepanek 7b94f257dd fix CORDIO not handling all adv set stop events 2021-06-09 18:50:17 +01:00
Martin Kojtal a8fd7b3f0e
Merge pull request #14693 from 16L-YT/master
Fix compilation errors with kvstore - TDB_INTERNAL option to store BLE bonding info
2021-06-09 16:35:56 +02:00
Paul Szczepanek c4585b977d clarify documentation for adv stop event 2021-06-09 13:56:39 +01:00
Paul Szczepanek 88ede0002d new error codes needed for adv sets 2021-06-09 13:54:19 +01:00
Martin Kojtal e377383f6d
Merge pull request #14672 from paul-szczepanek-arm/fix-advertising-start
BLE: Fix advertising start and stop
2021-06-09 10:38:34 +02:00
Paul Szczepanek 2d9a781dd8 fix typos and traces 2021-06-08 09:48:45 +01:00
Paul Szczepanek 0515408f21 add default BLE roles configuration in case no json present 2021-06-06 11:54:24 +01:00
Paul Szczepanek c51dc2f6cc set to pending only after issuing command 2021-06-02 12:06:43 +01:00
Paul Szczepanek b21ee6b0a0 add information about privacy refresh to config option 2021-06-02 12:06:12 +01:00
Paul Szczepanek 8ec7cfd820 start and stop adv sets in batches 2021-06-01 22:38:39 +01:00
Paul Szczepanek 0670470af6 add config for number of pending adv sets starting 2021-06-01 22:38:13 +01:00
Paul Szczepanek 8dedd43e7c add status to start and stop events 2021-06-01 22:37:43 +01:00
Paul Szczepanek e99741dd3d change function names 2021-06-01 15:45:08 +01:00
Abbas Bracken Ziad 8ee7fb8642 BLE: support non-low power devices 2021-05-27 10:11:05 +01:00
Anna Bridge 8b1cd98a8e
Merge pull request #14574 from noonfom/string-buffer-length
Fix string buffer length in UUID trace helper
2021-05-25 12:38:06 +01:00
Yahya Tawil ae31d20904
Merge branch 'master' into master 2021-05-24 20:04:41 +03:00
Yahya Tawil 6318e3a3b7
Solve the type error in one of the tracing msgs
Adapted from #14684
2021-05-22 11:49:49 +03:00
Yahya Tawil 997b523284
Add the missing definition for `as_entry` 2021-05-22 11:46:34 +03:00
Yahya Tawil 27e64a82b8
Add the missing mbed-storage libraries
Refer to #14660
2021-05-22 11:40:36 +03:00
George Beckstein 2578c5356b Fix missing `as_entry` method in KVStoreSecurityDb
This commit introduces changes to fix a missing method, `as_entry`, and traces that were not updated properly to work with this new method.

Prior to this fix, it was not possible to use KVStoreSecurityDb as the build would fail since the `as_entry` method was not declared in the class header file.

This bug was introduced with commit 957486e0eb in PR ARMmbed/mbed-os#14198
2021-05-19 16:38:13 -04:00