arduino/esp8266
1
0
Fork 0
mirror of https://github.com/esp8266/Arduino.git synced 2025-04-19 23:22:16 +03:00
Code

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

Browse Source 
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.
This commit is contained in:
Earle F. Philhower, III 2023-01-07 10:01:56 -08:00 committed by GitHub
parent a76852a991
commit c8dcdede34
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
libraries/ESP8266WebServer/examples/AdvancedWebServer/AdvancedWebServer.ino
View File

@ -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);
}