Decrease Stack Usage in wifiScanDoneCb()

chmorgan / libesphttpd

Libesphttpd - web server for ESP8266 / ESP32

Mozilla Public License 2.0

128 stars 45 forks source link

Closed tidklaas closed 6 years ago

tidklaas commented 6 years ago

cgiwifi.c/wifiScanDoneCb():

Use dynamically allocated buffer for retrieving the scan result set. Reduces risk of stack overflow in WiFi event task.
Improve handling of memory allocation errors, i.e. actually check if allocation succeeded.
Use strlcpy/memcpy when copying ssid/bssid from scan results.

This should eliminate the need for increasing WiFi event task stack size in issue #18 for now.

chmorgan commented 6 years ago

Thanks for the PR!