From d9a5b045b8384fcdef6200f2cdc293805957fe5e Mon Sep 17 00:00:00 2001 From: Edd Inglis Date: Fri, 11 Mar 2022 22:03:48 +0000 Subject: [PATCH] Catch a possible null-dereference --- libraries/ESP8266httpUpdate/src/ESP8266httpUpdate.cpp | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/libraries/ESP8266httpUpdate/src/ESP8266httpUpdate.cpp b/libraries/ESP8266httpUpdate/src/ESP8266httpUpdate.cpp index 741c701804..df709bdd2f 100755 --- a/libraries/ESP8266httpUpdate/src/ESP8266httpUpdate.cpp +++ b/libraries/ESP8266httpUpdate/src/ESP8266httpUpdate.cpp @@ -254,6 +254,12 @@ HTTPUpdateResult ESP8266HTTPUpdate::handleUpdate(HTTPClient& http, const String& } WiFiClient * tcp = http.getStreamPtr(); + if (!tcp) { + DEBUG_HTTP_UPDATE("[httpUpdate] WiFiClient connection unexpectedly absent\n"); + _setLastError(HTTPC_ERROR_CONNECTION_LOST); + http.end(); + return HTTP_UPDATE_FAILED; + } if (_closeConnectionsOnUpdate) { WiFiUDP::stopAll();