wenil/clawdmeter
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

Merge pull request #27 from orbisai0security/fix-ble-rx-buf-bounds-check

Browse Source 
fix: add buffer-length check in ble.cpp
This commit is contained in:
Hermann Björgvin
2026-05-21 19:52:00 +00:00
committed by GitHub
parent 09b5ba8c72 5079ea1af4
commit c25e89672a
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
firmware/src/ble.cpp
+1 -2
View File
@@ -126,8 +126,7 @@ class ServerCallbacks : public NimBLEServerCallbacks {
class RxCallbacks : public NimBLECharacteristicCallbacks { class RxCallbacks : public NimBLECharacteristicCallbacks {
void onWrite(NimBLECharacteristic* chr, NimBLEConnInfo& info) override { void onWrite(NimBLECharacteristic* chr, NimBLEConnInfo& info) override {
std::string val = chr->getValue(); std::string val = chr->getValue();
size_t len = val.length(); size_t len = std::min(val.length(), (size_t)(BLE_BUF_SIZE - 1));
if (len >= BLE_BUF_SIZE) len = BLE_BUF_SIZE - 1;
memcpy(rx_buf, val.c_str(), len); memcpy(rx_buf, val.c_str(), len);
rx_buf[len] = '\0'; rx_buf[len] = '\0';
data_ready = true; data_ready = true;