Remove 400b stack allocation from AdvWeb example (#8793)

The AdvancedWebServer.ino example allocated a 400 byte char array on the stack which, in the case of the example, will work but in general is a dangerous thing to show new users to try. Instead, use a StreamString to generate the string on the heap.
esp8266 · Jan 7, 2023 · c8dcded · c8dcded
1 parent a76852a
commit c8dcded
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/libraries/ESP8266WebServer/examples/AdvancedWebServer/AdvancedWebServer.ino b/libraries/ESP8266WebServer/examples/AdvancedWebServer/AdvancedWebServer.ino
@@ -32,6 +32,7 @@
 #include <WiFiClient.h>
 #include <ESP8266WebServer.h>
 #include <ESP8266mDNS.h>
+#include <StreamString.h>
 
 #ifndef STASSID
 #define STASSID "your-ssid"
@@ -47,14 +48,14 @@ const int led = 13;
 
 void handleRoot() {
   digitalWrite(led, 1);
-  char temp[400];
   int sec = millis() / 1000;
   int min = sec / 60;
   int hr = min / 60;
 
-  snprintf(temp, 400,
-
-           "<html>\
+  StreamString temp;
+  temp.reserve(500);  // Preallocate a large chunk to avoid memory fragmentation
+  temp.printf("\
+<html>\
   <head>\
     <meta http-equiv='refresh' content='5'/>\
     <title>ESP8266 Demo</title>\
@@ -68,9 +69,8 @@ void handleRoot() {
     <img src=\"/test.svg\" />\
   </body>\
 </html>",
-
-           hr, min % 60, sec % 60);
-  server.send(200, "text/html", temp);
+              hr, min % 60, sec % 60);
+  server.send(200, "text/html", temp.c_str());
   digitalWrite(led, 0);
 }