fix(ble): Modify size of rssi in BLEAdvertisedDevice.toString()#12058
fix(ble): Modify size of rssi in BLEAdvertisedDevice.toString()#12058me-no-dev merged 1 commit intoespressif:masterfrom
Conversation
👋 Hello Oros42, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Test Results 76 files 76 suites 17m 46s ⏱️ For more details on these failures, see this check. Results for commit 77e678e. ♻️ This comment has been updated with latest results. |
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Description of Change
In BLEAdvertisedDevice.toString(), modify the rssi size to avoid truncation.
Why
When rssi ==
-102, it print :Advertised Device: Name: , Address: 12:34:56:78:9a:bc, manufacturer data: 123456789, txPower: 12, rssi: -10because snprintf add
\0at the end of the array.So the size of
-102\0is 5 not 4.