Bicycle_Market_Research/DoorCounter
Public Access
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: config_save_wifi — always write both credentials

Browse Source 
Replace short-circuit boolean evaluation of putString return values with
separate size_t variables so both writes always execute regardless of
whether the first succeeds.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Peter Woolery
2026-04-13 14:05:26 -07:00
parent d5afd0bd87
commit 74bff0912b
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
firmware/src/config.cpp
View File

@@ -25,10 +25,10 @@ bool config_load(DeviceConfig& cfg) {
bool config_save_wifi(const String& ssid, const String& pass) { bool config_save_wifi(const String& ssid, const String& pass) {
Preferences prefs; Preferences prefs;
prefs.begin(NS, false); prefs.begin(NS, false);
bool ok = prefs.putString("wifi_ssid", ssid) && size_t r1 = prefs.putString("wifi_ssid", ssid);
prefs.putString("wifi_pass", pass); size_t r2 = prefs.putString("wifi_pass", pass);
prefs.end(); prefs.end();
return ok; return (r1 > 0) && (r2 > 0);
} }
bool config_has_wifi() { bool config_has_wifi() {