ota: shorten error code text for 1.x displays
What the developer wrote
Authored by Jon Griffiths
ota: shorten error code text for 1.x displays
What changed, and why it matters
This commit simply shortens the text shown on screen when an over-the-air (OTA) firmware update fails on older 1.x displays. It changes strings like 'OTA_ERR_SETUP' to 'ERR_SETUP' so they fit better on small screens. There is no security change.
No security action needed. Treat as a normal UI/UX maintenance change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch modifies ota_get_status_text() in main/process/ota_util.c to return shorter error-code strings by removing the ‘OTA_’ prefix from every OTA error label. The function’s behavior, return values, and call sites are unchanged; only the human-readable display text is shortened. This is a UI/string-length adjustment for 1.x device displays.
Changed components
main/process/ota_util.cInspect captured patch +15 / −15
diff --git a/main/process/ota_util.c b/main/process/ota_util.c
index b45f0c6..2017a9e 100644
--- a/main/process/ota_util.c
+++ b/main/process/ota_util.c
@@ -28,35 +28,35 @@ static const char* ota_get_status_text(const ota_status_t status)
case OTA_SUCCESS:
return "OK";
case OTA_ERR_SETUP:
- return "OTA_ERR_SETUP";
+ return "ERR_SETUP";
case OTA_ERR_INIT:
- return "OTA_ERR_INIT";
+ return "ERR_INIT";
case OTA_ERR_BADPARTITION:
- return "OTA_ERR_BADPARTITION";
+ return "ERR_BADPARTITION";
case OTA_ERR_DECOMPRESS:
- return "OTA_ERR_DECOMPRESS";
+ return "ERR_DECOMPRESS";
case OTA_ERR_WRITE:
- return "OTA_ERR_WRITE";
+ return "ERR_WRITE";
case OTA_ERR_FINISH:
- return "OTA_ERR_FINISH";
+ return "ERR_FINISH";
case OTA_ERR_SETPARTITION:
- return "OTA_ERR_SETPARTITION";
+ return "ERR_SETPARTITION";
case OTA_ERR_BADDATA:
- return "OTA_ERR_BADDATA";
+ return "ERR_BADDATA";
case OTA_ERR_NODOWNGRADE:
- return "OTA_ERR_NODOWNGRADE";
+ return "ERR_NODOWNGRADE";
case OTA_ERR_INVALIDFW:
- return "OTA_ERR_INVALIDFW";
+ return "ERR_INVALIDFW";
case OTA_ERR_USERDECLINED:
- return "OTA_ERR_USERDECLINED";
+ return "ERR_USERDECLINED";
case OTA_ERR_BADHASH:
- return "OTA_ERR_BADHASH";
+ return "ERR_BADHASH";
case OTA_ERR_PATCH:
- return "OTA_ERR_PATCH";
+ return "ERR_PATCH";
case OTA_ERR_PROTOCOL:
- return "OTA_ERR_PROTOCOL";
+ return "ERR_PROTOCOL";
default:
- return "OTA_ERR_UNKNOWN";
+ return "ERR_UNKNOWN";
}
}
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.