Bugfix preventing crash during parameter file download

Author: fredlcore

BSB_LAN/BSB_LAN.ino

@@ -5276,13 +5276,12 @@ void loop() {
                           printToWebClient(" - ");
                           printToWebClient_prognrdescaddr();
                           printFmtToWebClient("\r\n0x%08X\r\n", c);
-                          int outBufLen = strlen(outBuf);
-                          bin2hex(outBuf + outBufLen, tx_msg, tx_msg[bus->getLen_idx()]+bus->getBusType(), ' ');
-                          printToWebClient(outBuf + outBufLen);
+                          bin2hex(outBuf, tx_msg, tx_msg[bus->getLen_idx()]+bus->getBusType(), ' ');
+                          printToWebClient(outBuf);
                           printToWebClient("\r\n");
-                          bin2hex(outBuf + outBufLen, msg, msg[bus->getLen_idx()]+bus->getBusType(), ' ');
-                          printToWebClient(outBuf + outBufLen);
-                          outBuf[outBufLen] = 0;
+                          bin2hex(outBuf, msg, msg[bus->getLen_idx()]+bus->getBusType(), ' ');
+                          printToWebClient(outBuf);
+                          outBuf[0] = 0;
                           printToWebClient("\r\n");
                         }
                         forcedflushToWebClient(); //browser will build page immediately
@@ -5297,7 +5296,6 @@ void loop() {
             printToWebClient("\r\nComplete dump:\r\n");
             c = 0;
             unsigned long timeout = 0;
-            int outBufLen = strlen(outBuf);
 
             uint32_t cmd_ids[] = {0x053D0001, 0x053D0064, 0x05000066, 0x05000068, 0x05000069, 0x06000066, 0x05000094, 0x05000095, 0x05000096, 0x05000097};
             for (uint i = 0; i<sizeof(cmd_ids)/sizeof(cmd_ids[0]); i++) {
@@ -5309,9 +5307,8 @@ void loop() {
               }
               printTelegram(tx_msg, -1);
               printTelegram(msg, -1);
-              outBufLen = strlen(outBuf);
-              bin2hex(outBuf + outBufLen, msg, msg[bus->getLen_idx()]+bus->getBusType(), ' ');
-              printToWebClient(outBuf + outBufLen);
+              bin2hex(outBuf, msg, msg[bus->getLen_idx()]+bus->getBusType(), ' ');
+              printToWebClient(outBuf);
               printToWebClient("\r\n");
             }
             flushToWebClient();
@@ -5404,13 +5401,12 @@ void loop() {
                   }
                 }
                 if (valid_response) {
-                  outBufLen = strlen(outBuf);
-                  int hex_len = bin2hex(outBuf + outBufLen, msg, msg[bus->getLen_idx()] + bus->getBusType(), ' ');
+                  int hex_len = bin2hex(outBuf, msg, msg[bus->getLen_idx()] + bus->getBusType(), ' ');
                   if (hex_len == 0) {
                     printToWebClient("Invalid telegram received: ");
-                    bin2hex(outBuf + outBufLen, msg, 10, ' ');  // Only first 10 bytes printed
+                    bin2hex(outBuf, msg, 10, ' ');  // Only first 10 bytes printed
                   }
-                  printToWebClient(outBuf + outBufLen);
+                  printToWebClient(outBuf);
                 } else {
                   printToWebClient("[Timeout] No valid response after retries.\r\n");
                 }
@@ -5441,21 +5437,19 @@ void loop() {
 
                 int length = msg[bus->getLen_idx()] + bus->getBusType();
 
-                outBufLen = strlen(outBuf);
                 if (length == 0 || length < 10) {
                   printToWebClient("Invalid telegram received: ");
-                  bin2hex(outBuf + outBufLen, msg, 10, ' ');  // Only first 10 bytes printed
-                  printToWebClient(outBuf + outBufLen);
+                  bin2hex(outBuf, msg, 10, ' ');  // Only first 10 bytes printed
+                  printToWebClient(outBuf);
                   printToWebClient("\r\n");
                 } else {
-                  bin2hex(outBuf + outBufLen, msg, length, ' ');
-                  printToWebClient(outBuf + outBufLen);
+                  bin2hex(outBuf, msg, length, ' ');
+                  printToWebClient(outBuf);
                   printToWebClient("\r\n");
                 }
                 flushToWebClient();
               }
-              outBufLen = strlen(outBuf);
-              outBuf[outBufLen] = 0;
+              outBuf[0] = 0;
             } else {
               printlnToDebug("No response to dump request:");
               bus->print(tx_msg);