Fix: Clear parser buffers to prevent random numbers if no data was re…

…ceived
hoylabs · Jul 31, 2023 · 5bbc67b · 5bbc67b
1 parent 2eeb742
commit 5bbc67b
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 0 deletions.
diff --git a/lib/Hoymiles/src/parser/AlarmLogParser.cpp b/lib/Hoymiles/src/parser/AlarmLogParser.cpp
@@ -96,6 +96,7 @@ AlarmLogParser::AlarmLogParser()
 {
     _xSemaphore = xSemaphoreCreateMutex();
     HOY_SEMAPHORE_GIVE(); // release before first use
+    clearBuffer();
 }
 
 void AlarmLogParser::clearBuffer()

diff --git a/lib/Hoymiles/src/parser/DevInfoParser.cpp b/lib/Hoymiles/src/parser/DevInfoParser.cpp
@@ -56,6 +56,8 @@ DevInfoParser::DevInfoParser()
 {
     _xSemaphore = xSemaphoreCreateMutex();
     HOY_SEMAPHORE_GIVE(); // release before first use
+    clearBufferSimple();
+    clearBufferAll();
 }
 
 void DevInfoParser::clearBufferAll()

diff --git a/lib/Hoymiles/src/parser/StatisticsParser.cpp b/lib/Hoymiles/src/parser/StatisticsParser.cpp
@@ -38,6 +38,7 @@ StatisticsParser::StatisticsParser()
 {
     _xSemaphore = xSemaphoreCreateMutex();
     HOY_SEMAPHORE_GIVE(); // release before first use
+    clearBuffer();
 }
 
 void StatisticsParser::setByteAssignment(const byteAssign_t* byteAssignment, uint8_t size)

diff --git a/lib/Hoymiles/src/parser/SystemConfigParaParser.cpp b/lib/Hoymiles/src/parser/SystemConfigParaParser.cpp
@@ -16,6 +16,7 @@ SystemConfigParaParser::SystemConfigParaParser()
 {
     _xSemaphore = xSemaphoreCreateMutex();
     HOY_SEMAPHORE_GIVE(); // release before first use
+    clearBuffer();
 }
 
 void SystemConfigParaParser::clearBuffer()