Merge pull request chrisjoyce911#144 from BrentIO/Fix-reading-PEM-issue

Fixed issue where rootcastr was empty because size() hadn't yet been …
tobozo · Apr 4, 2024 · c8573d1 · c8573d1
2 parents facd0ee + 7a5c177
commit c8573d1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/esp32FOTA.cpp b/src/esp32FOTA.cpp
@@ -364,14 +364,14 @@ bool esp32FOTA::setupHTTP( const char* url )
                 log_e("A strict security context has been set but no RootCA was provided");
                 return false;
             }
-            rootcastr = _cfg.root_ca->get();
             if( _cfg.root_ca->size() == 0 ) {
                 log_e("A strict security context has been set but an empty RootCA was provided");
-                log_e("rootcastr=%s", rootcastr);
                 return false;
             }
+            rootcastr = _cfg.root_ca->get();
             if( !rootcastr ) {
                 log_e("Unable to get RootCA, aborting");
+                log_e("rootcastr=%s", rootcastr);
                 return false;
             }
             log_d("Loading root_ca.pem");