What the developer wrote
Authored by Jon Griffiths
gui: add missing header
What changed, and why it matters
This is a one-line change that adds a missing C header file include (#include "button_events.h") to a source file in the Blockstream Jade hardware wallet firmware. It is a routine build/fix commit with no apparent security relevance.
No security action required; treat as a normal build hygiene fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit adds an explicit include for button_events.h in main/gui.c. Without this header, the file may have relied on transitive includes or forward declarations. The change ensures the compiler sees the correct declarations for button event types/functions used in the GUI code. There is no functional code change, no bounds-checking fix, no cryptographic change, and no privilege/authorization change.
Changed components
main/gui.cInspect captured patch +1 / −0
diff --git a/main/gui.c b/main/gui.c
index 3851c38..a33bffb 100644
--- a/main/gui.c
+++ b/main/gui.c
@@ -10,6 +10,7 @@
#include "display.h"
#include "ble/ble.h"
+#include "button_events.h"
#include "gui.h"
#include "idletimer.h"
#include "jade_assert.h"
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.