From 5e487d6edc1c1331dfe41eec5565f0aab9cd2e04 Mon Sep 17 00:00:00 2001
From: Shawn A <spam@rumental.com>
Date: Tue, 6 Mar 2018 16:42:54 -0600
Subject: [PATCH] remove preallocation of params memory

---
 WiFiManager.cpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/WiFiManager.cpp b/WiFiManager.cpp
index 8e92d0e..c73b345 100644
--- a/WiFiManager.cpp
+++ b/WiFiManager.cpp
@@ -105,10 +105,16 @@ bool WiFiManager::addParameter(WiFiManagerParameter *p) {
      }
   }
 
+  if(_params == NULL){
+    DEBUG_WM("allocating params bytes:",_max_params * sizeof(WiFiManagerParameter*));        
+    _params = (WiFiManagerParameter**)malloc(_max_params * sizeof(WiFiManagerParameter*));
+  }
+
   if(_paramsCount == _max_params){
     // resize the params array by increment of WIFI_MANAGER_MAX_PARAMS
     _max_params += WIFI_MANAGER_MAX_PARAMS;
     DEBUG_WM(F("Updated _max_params:"),_max_params);
+    DEBUG_WM("re-allocating params bytes:",_max_params * sizeof(WiFiManagerParameter*));    
     WiFiManagerParameter** new_params = (WiFiManagerParameter**)realloc(_params, _max_params * sizeof(WiFiManagerParameter*));
     // DEBUG_WM(WIFI_MANAGER_MAX_PARAMS);
     // DEBUG_WM(_paramsCount);
@@ -146,11 +152,7 @@ void WiFiManager::WiFiManagerInit(){
   WiFi.persistent(false); // disable persistent so scannetworks and mode switching do not cause overwrites
   
   setMenu(_menuIds);
-  //parameters
-  // @todo belongs to wifimanagerparameter
   _max_params = WIFI_MANAGER_MAX_PARAMS;
-  DEBUG_WM("allocating",_max_params * sizeof(WiFiManagerParameter*));
-  _params = (WiFiManagerParameter**)malloc(_max_params * sizeof(WiFiManagerParameter*));
 }
 
 // destructor