ElixirForge/WiFiManager
1
0
Fork 0
mirror of https://github.com/alexhopeoconnor/WiFiManager.git synced 2026-10-13 14:51:21 +10:00
Code Releases Activity

Bug fix for problem discovered when setting length to 1 for additional parameters

Browse Source 
The initialization of the buffer holding the value stopped before the last character and storage did not take into account the extra-byte for end of string.
This commit is contained in:
Laurent Louf
2018-02-06 14:12:13 +01:00
parent a3a04f1f9a
commit 6aaa83a266
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
WiFiManager.cpp
+2 -2
View File
@@ -34,7 +34,7 @@ void WiFiManagerParameter::init(const char *id, const char *placeholder, const c
_placeholder = placeholder;
_length = length;
_value = new char[length + 1];
for (int i = 0; i < length; i++) {
for (int i = 0; i < length + 1; i++) {
_value[i] = 0;
}
if (defaultValue != NULL) {
@@ -597,7 +597,7 @@ void WiFiManager::handleWifiSave() {
//read parameter
String value = server->arg(_params[i]->getID()).c_str();
//store it in array
value.toCharArray(_params[i]->_value, _params[i]->_length);
value.toCharArray(_params[i]->_value, _params[i]->_length + 1);
DEBUG_WM(F("Parameter"));
DEBUG_WM(_params[i]->getID());
DEBUG_WM(value);