fix: prevent stack overflow when logging error occurs

Author: marcelkottmann

components/esp32-javascript/modules/esp32-javascript/filelogging.js

@@ -54,12 +54,18 @@ function cleanupOldLogs() {
     }
 }
 global.el_flushLogBuffer = function () {
+    var swap = global.logBuffer; // swap to prevent endless loop when error occurs in this method
+    global.logBuffer = new stringbuffer_1.StringBuffer();
     var logFile = exports.FILE_LOGGING_DIRECTORY + "/logs-" + getLogFileNumber() + ".txt";
-    var ret = appendFile(logFile, global.logBuffer.toString());
-    if (ret < 0) {
-        console.error("Could not flush log file. Space exceeded?");
+    try {
+        var ret = appendFile(logFile, swap.toString());
+        if (ret < 0) {
+            console.error("Could not flush log file. Space exceeded?");
+        }
+    }
+    catch (error) {
+        console.error("Could not open log file. Space exceeded?");
     }
-    global.logBuffer = new stringbuffer_1.StringBuffer();
     if (fileSize(logFile) > exports.LOG_FILE_SIZE_LIMIT) {
         logFileNumber++;
     }

components/esp32-javascript/modules/esp32-javascript/filelogging.ts

@@ -57,13 +57,19 @@ function cleanupOldLogs() {
 }
 
 global.el_flushLogBuffer = function (): void {
+  const swap = global.logBuffer; // swap to prevent endless loop when error occurs in this method
+  global.logBuffer = new StringBuffer();
+
   const logFile = `${FILE_LOGGING_DIRECTORY}/logs-${getLogFileNumber()}.txt`;
-  const ret = appendFile(logFile, global.logBuffer.toString());
-  if (ret < 0) {
-    console.error("Could not flush log file. Space exceeded?");
+  try {
+    const ret = appendFile(logFile, swap.toString());
+    if (ret < 0) {
+      console.error("Could not flush log file. Space exceeded?");
+    }
+  } catch (error) {
+    console.error("Could not open log file. Space exceeded?");
   }
 
-  global.logBuffer = new StringBuffer();
   if (fileSize(logFile) > LOG_FILE_SIZE_LIMIT) {
     logFileNumber++;
   }