From 564b277f6e828bd0d2118b79279f9c3b067b9185 Mon Sep 17 00:00:00 2001 From: Linar Yusupov Date: Wed, 5 Mar 2025 21:20:46 +0300 Subject: [PATCH] [RF69] a fix for invalid sync word length setting --- src/modules/RF69/RF69.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/modules/RF69/RF69.cpp b/src/modules/RF69/RF69.cpp index f1fdd7fc..aa162146 100644 --- a/src/modules/RF69/RF69.cpp +++ b/src/modules/RF69/RF69.cpp @@ -696,7 +696,7 @@ int16_t RF69::setOutputPower(int8_t pwr, bool highPower) { int16_t RF69::setSyncWord(const uint8_t* syncWord, size_t len, uint8_t maxErrBits) { // check constraints - if((maxErrBits > 7) || (len > 8)) { + if((maxErrBits > 7) || (len == 0) || (len > 8)) { return(RADIOLIB_ERR_INVALID_SYNC_WORD); } @@ -712,12 +712,12 @@ int16_t RF69::setSyncWord(const uint8_t* syncWord, size_t len, uint8_t maxErrBit RADIOLIB_ASSERT(state); // set the length - state = this->mod->SPIsetRegValue(RADIOLIB_RF69_REG_SYNC_CONFIG, len, 5, 3); + state = this->mod->SPIsetRegValue(RADIOLIB_RF69_REG_SYNC_CONFIG, len-1, 5, 3); // set sync word register this->mod->SPIwriteRegisterBurst(RADIOLIB_RF69_REG_SYNC_VALUE_1, syncWord, len); if(state == RADIOLIB_ERR_NONE) { - this->syncWordLength = len; + this->syncWordLength = len-1; } return(state);