BS
← All projectsBlockstream

Blockstream Jade

Firmware for Jade and Jade Plus Bitcoin hardware wallets.

BitcoinHardware walletsElevated watch
Repository coverage

502 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

106security candidates69second-pass queue437AI analyses
25commits · 30 days
67commits · 60 days
220commits · 180 days
417commits · 365 days
Backfill bands
Aug 5 → Feb 6216 seen38 candidatesComplete
Feb 6 → Jun 6131 seen9 candidatesComplete
Jun 6 → Jul 657 seen10 candidatesComplete
Jul 6 → Aug 510 seen0 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

57/100 average clarity
32Strong · 80–100
182Adequate · 60–79
250Thin · 40–59
38Opaque · 0–39
5security candidates with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
Jon Griffiths31357282157
Mike Tolkachev431438152
Daniel Newton771668055
Mihailo Milenkovic878057
roderico333060
Valerio Vaccaro1526054
Jamie C. Driver525053
Gustavo Cateim917078
Austin-Fulbright414060
quantpoet111072
cmiguelrn111045
tokyovyse111063
Analysis record

Published AI watches

Last scanned 35 minutes ago

Informational 20 AI analysisMessage 65 · Adequate
BS BlockstreamBlockstream Jade BitcoinHardware wallets

fmt: fix and prevent future use of unsupported format spec "zu"

This commit fixes the use of the "%zu" printf format specifier, which is not supported by the ESP32 toolchain used in this project. The unsupported format could cause log messages and user-facing strings to display incorrect or garbage val…

Use of unsupported printf format specifier %zu on embedded targetPotential for malformed log/UI output due to undefined format behaviorAddition of CI guard to prevent recurrence
26042182by Jon Griffiths+21−145 files
No security note in commit
Low 33 AI analysisMessage 30 · Opaque
BS BlockstreamBlockstream Jade BitcoinHardware wallets

otp: release md context on failure and clean temporary buffers

This commit fixes cleanup in the one-time-password (OTP) code. Previously, if setting up the cryptographic hash context failed, the code could return without freeing that context, and temporary buffers holding sensitive counter/HMAC data w…

Resource leak fix: mbedtls_md_free() now called on prepare_md_ctx failure pathSensitive buffer zeroization: wally_bzero() added for rcnt_buf and hmacControl-flow change from macro-based early return to explicit success checks
6e2cb6b6by Mike Tolkachev+26−141 file
No security note in commit
Low 47 AI analysisMessage 25 · Opaque
BS BlockstreamBlockstream Jade BitcoinHardware wallets

identity: clean temporary hash buffer

This commit fixes a small but real security hygiene issue in Blockstream Jade, a hardware wallet. A temporary buffer that holds a sensitive cryptographic hash (HMAC-SHA512 output used to derive child private keys) was not marked as sensiti…

Sensitive memory not marked for clearing (missing SENSITIVE_PUSH/POP)Buffer holds HMAC-SHA512 output used in BIP32 hardened child key derivationPatch adds explicit sensitive-memory lifecycle management
b5c35779by Mike Tolkachev+2−01 file
No security note in commit
Low 36 AI analysisMessage 15 · Opaque
BS BlockstreamBlockstream Jade BitcoinHardware wallets

qrmode: clean temporary buffer

This commit adds cleanup of a temporary memory buffer used when turning data into QR codes on the Blockstream Jade hardware wallet. The buffer could previously remain in memory after use, potentially leaving sensitive QR data behind for la…

Sensitive temporary buffer not cleared before fixUse of SENSITIVE_PUSH/SENSITIVE_POP memory-clearing macrosQR payload data may include private or confidential material
34f68c20by Mike Tolkachev+2−01 file
No security note in commit
Low 43 AI analysisMessage 50 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

pinserver: retain unit private key on url change

This commit changes how Blockstream Jade handles its PIN server settings. Previously, changing the PIN server web address (URL) also wiped the device's private PIN encryption key, forcing the device to create a fresh one. The patch keeps t…

PIN private key lifecycle change: key retained on URL-only pinserver updatePublic-key change still triggers private-key erasure/regenerationDebug-clean and full reset still erase private key
ddcd93a3by Mike Tolkachev+38−324 files
No security note in commit
Informational 15 AI analysisMessage 45 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

add more boards and descriptions

This commit is a documentation update for the do-it-yourself (DIY) hardware section of the Blockstream Jade project. It adds descriptions and photos for additional supported circuit boards and updates image file references. There are no co…

f4fb150dby Valerio Vaccaro+65−1625 files
No security note in commit
Moderate 63 AI analysisMessage 50 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

process: limit array sizes in sign_tx, register_multisig and sign_bip85_digests

This commit adds size limits to several message-handling functions in the Blockstream Jade hardware wallet firmware. It caps how many asset records, transaction inputs/outputs, multisig signers, and BIP85 RSA digests a single RPC message c…

Adds explicit upper bounds on attacker-controllable array sizesRejects zero-input/zero-output transactions before further processingMoves BIP85 digest count validation before memory allocation
e87f26bfby Daniel Newton+251−8114 files
No security note in commit
Informational 15 AI analysisMessage 28 · Opaque
BS BlockstreamBlockstream Jade BitcoinHardware wallets

Update CHANGELOG.md

This commit only updates the project's CHANGELOG.md file to add release notes for version 1.0.41. It does not change any source code, build scripts, or configuration. The changelog text itself states that version 1.0.41 contains important …

Changelog describes prior security fixes in release 1.0.41Mentions external security researchers and tools (Loupe, Kvazar)References security-relevant improvements: stack protection, entropy, secure version, memory scrubbing, thread safety
dd45a7dcby Jon Griffiths+49−01 file
Vendor flagged security relevance
Informational 15 AI analysisMessage 35 · Opaque
BS BlockstreamBlockstream Jade BitcoinHardware wallets

jadepy: update jadepy release

This commit simply bumps the version of a Python dependency called jade-client from 1.0.40 to 1.0.41, along with its matching cryptographic hashes and a fresh PGP signature. There is no code change visible here, and nothing in the commit m…

75eeada4by Jon Griffiths+19−192 files
No security note in commit
Informational 15 AI analysisMessage 45 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

consistency: fix docs/comment/script typos

This commit fixes five spelling mistakes in documentation, code comments, and a script comment. None of the changes alter program behavior or affect security.

9ea7eed3by Jon Griffiths+5−55 files
No security note in commit
Informational 17 AI analysisMessage 50 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

esp_lcd: vendor the esp_lcd component (ESP-IDF v5.5.4)

This commit copies the ESP-IDF 'esp_lcd' display-driver component directly into the Blockstream Jade firmware repository (a process called 'vendoring') at version 5.5.4. It adds about 8,700 lines of display-interface code for LCD panels us…

Large vendored third-party driver import (+8,718 lines, 43 files)Driver code runs in interrupt context and uses DMA, which historically carries memory-safety and cache-coherency risksNo explicit security fix or bug-fix narrative in commit title/message
d89207bfby Daniel Newton+8718−043 files
No security note in commit
Informational 21 AI analysisMessage 57 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

build: "fakeprod" prod-like development mode

This commit adds an official developer-only 'fakeprod' build mode for Blockstream Jade hardware wallets. It deliberately weakens several security settings (single signature, committed private keys, readable flash-encryption key, enabled JT…

In-tree private keys committed to repository (tools/fakeprod_v2.pem, tools/fakeprod_attest.pem)Secure boot minimum signatures reduced from 2 to 1Flash encryption set to DEVELOPMENT mode with key readable
cecedc76by Daniel Newton+350−59 files
Vendor flagged security relevance
Low 45 AI analysisMessage 45 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

urldecode: add validation for URL encoding

This commit hardens a URL-decoding helper in Blockstream Jade's firmware. It adds validation so malformed percent-encoding, embedded null bytes, control characters, and non-ASCII bytes are rejected instead of being silently decoded. A new …

Input validation added for percent-encoded sequencesRejection of embedded NUL bytes and control characters in decoded outputRejection of non-ASCII decoded bytes (>=0x80 and DEL 0x7f)
dd0d699dby Mike Tolkachev+100−272 files
No security note in commit
Low 26 AI analysisMessage 73 · Adequate
BS BlockstreamBlockstream Jade BitcoinHardware wallets

qr: split datastream data into its own allocation, reorder alloc/free calls

This change rewrites how a small QR-code scanning library allocates memory. Instead of one big 16 KB block, it now uses two separate 8 KB blocks, and it frees them in the reverse order they were allocated. The stated goal is to avoid memor…

Memory allocation pattern changed from single large allocation to two smaller allocationsAllocation/free ordering changed to LIFO-style reverse pairingPointer field added to struct that previously held an inline array
43640f8cby Jon Griffiths+24−153 files
No security note in commit
Low 46 AI analysisMessage 45 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

psbt: check output amount presence before reading

This commit tightens how Blockstream Jade reads output amounts from a Partially Signed Bitcoin Transaction (PSBT). Before, the code tried to read the amount directly without first confirming it was actually present. Now it checks whether t…

Missing-field check added before sensitive value readPSBT output amount parsing hardenedDefensive validation of libwally presence flags
9f2b916dby Jon Griffiths+2−11 file
No security note in commit
Moderate 59 AI analysisMessage 50 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

psbt: require that all psbt bytes are consumed when parsing

This commit tightens how Blockstream Jade verifies a Bitcoin transaction file (PSBT) before signing. It now requires that every byte in the file is part of the transaction, with no leftover or trailing data allowed. Previously, extra bytes…

Parsing now rejects trailing/unconsumed bytes in PSBT inputChange is in the signing code path, directly affecting what the hardware wallet will signUse of stricter libwally-core parse flag (COMPLETE) indicates parser hardening
605d86acby Jon Griffiths+1−11 file
No security note in commit
Informational 15 AI analysisMessage 68 · Adequate
BS BlockstreamBlockstream Jade BitcoinHardware wallets

hardware: add Jade Plus design files

This commit reorganizes existing hardware design files for the original Jade device into a new subfolder and adds the open-source hardware design files for a new product, the Jade Plus. It does not change any software, firmware, or device …

5ad85661by bitcoinhelp+2573−122520 files
No security note in commit
Informational 15 AI analysisMessage 55 · Thin
BS BlockstreamBlockstream Jade BitcoinHardware wallets

tests: migrate low level tests

This commit only moves and reorganizes existing test code. It adds a new helper to check whether the current transport is a specific type, moves a shared wait() function into a common test package file, and migrates a large set of low-leve…

767006a0by Valerio Vaccaro+1580−54 files
No security note in commit
Low 27 AI analysisMessage 68 · Adequate
BS BlockstreamBlockstream Jade BitcoinHardware wallets

mnemonic: restore recovery phrase by word number

This commit adds a new way to restore a Bitcoin wallet on a Blockstream Jade hardware device: instead of typing each recovery word, users can enter each word's one-based BIP39 number (for example, '1' for 'abandon'). The change also refact…

New user-input path for seed recovery (word numbers instead of words)Refactoring of mnemonic input/validation helpersAddition of SENSITIVE_PUSH/POP for local mnemonic buffers
99927491by roderico+580−3183 files
No security note in commit
Informational 18 AI analysisMessage 68 · Adequate
BS BlockstreamBlockstream Jade BitcoinHardware wallets

ui: support bounded variable-length digit entry

This commit refactors the on-screen digit-entry keypad in Blockstream Jade so callers can ask for shorter number fields (for example 4 digits instead of 6) and optionally cap the entered value. The existing 6-digit PIN and index entry beha…

New bounded numeric input path may prevent out-of-range values in future callersRefactoring centralizes input validation rather than relying on callersNo buffer-size changes observed; digit array remains DIGIT_ENTRY_SIZE (6) elements
69627745by roderico+128−332 files
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

AI review queuedusb: simplify handle_ota_reply logicby Jon Griffiths · 1e51b96c · Oct 23, 2025 · 1 fileMessage 35 · OpaqueLow 34Details
Commit message · Jon Griffiths

usb: simplify handle_ota_reply logic

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Low 34/100

This commit simplifies how a Blockstream Jade hardware wallet handles an over-the-air (OTA) firmware update reply received over USB. Previously, the code checked whether a CBOR boolean 'result' field existed and only updated the success flag if it did. Now it updates the success flag regardless of whether the field is present, and no longer treats a missing 'result' as a failure. This could mean a malformed or unexpected reply leaves the success flag set to its previous value rather than being explicitly cleared, potentially making an unsuccessful update appear successful.

Lower-priorityusb: avoid memory allocations for ota chunk databy Jon Griffiths · f0ebdd09 · Oct 23, 2025 · 1 fileMessage 45 · ThinInformational 23Details
Commit message · Jon Griffiths

usb: avoid memory allocations for ota chunk data

Prevents fragmentation.

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 23/100

This commit changes how firmware-update (OTA) messages are passed internally in the Blockstream Jade hardware wallet's USB code. Instead of using a helper that allocates extra memory and copies the message, the code now embeds a one-byte source tag at the start of a single fixed buffer and passes that buffer directly. The stated goal is to reduce memory fragmentation. The change itself does not obviously fix a security bug, but it removes a memory-allocation path during firmware updates, which could indirectly reduce the chance of update failures or instability caused by low memory.

Lower-priorityusb: delete impossible-to-reach codeby Jon Griffiths · d550d332 · Oct 23, 2025 · 1 fileMessage 45 · ThinInformational 12Details
Commit message · Jon Griffiths

usb: delete impossible-to-reach code

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 12/100

This commit removes four lines of code from the USB mass-storage handling task. The deleted branch checked for two event flags that the surrounding logic already handles earlier, so it could never actually run. There is no indication this change fixes a security bug; it is a routine cleanup of dead code.

Lower-priorityusb: rename the interface mutex to reflect it is a mutexby Jon Griffiths · 25598dc3 · Oct 23, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Jon Griffiths

usb: rename the interface mutex to reflect it is a mutex

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit is purely a rename: a variable called 'interface_semaphore' is renamed to 'interface_mutex' throughout the code because it is actually a mutex, not a semaphore. No behavior changes, no security fix.

Lower-prioritydescriptor: early-reject descriptors with too many signersby Jon Griffiths · 72cc31db · Oct 23, 2025 · 2 filesMessage 50 · ThinModerate 59Details
Commit message · Jon Griffiths

descriptor: early-reject descriptors with too many signers

Reported-by: DARKNAVY <X@DarkNavyOrg>

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Moderate 59/100

This update fixes a boundary check in how Blockstream Jade handles wallet descriptors (the instructions that say how many keys are needed to spend funds). Previously, the device accepted descriptors with one more signer than its internal limit, which could cause memory corruption or unexpected behavior. The fix rejects oversized descriptors earlier and adds tests to confirm the boundary.

Lower-priorityusb: set event bits atomicallyby Jon Griffiths · 53bc9c8d · Oct 23, 2025 · 1 fileMessage 45 · ThinLow 31Details
Commit message · Jon Griffiths

usb: set event bits atomically

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Low 31/100

This commit fixes a small race condition in how USB event flags are set. Previously, two related event flags could be set in separate, non-atomic calls, meaning an observer might see only one flag update and miss the other. The patch combines the flags and sets them in a single atomic operation, ensuring consistent state. There is no direct evidence this is exploitable for security harm, but race conditions in event handling can theoretically lead to inconsistent USB host state.

Lower-priorityusb: don't hold the callback mutex during the callbackby Jon Griffiths · 21229c99 · Oct 23, 2025 · 1 fileMessage 73 · AdequateLow 44Details
Commit message · Jon Griffiths

usb: don't hold the callback mutex during the callback

Note that the callback is called via trigger_event() from a single thread
only and so holding the mutex while calling it is pointless. If the callback
triggers code which attempts to unset the callback (for example, during error
handling), then this will deadlock.

Instead, use the mutex only to protect reading and writing the callback and
context variables themselves.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 44/100

This commit fixes a potential deadlock in the Blockstream Jade hardware wallet's USB mass-storage code. Previously, a protective lock was held while calling a user-provided callback function. If that callback tried to unregister itself (for example, during error handling), it would wait forever for the lock that was already held, freezing the USB task. The fix copies the callback details under the lock, releases the lock, and only then invokes the callback. It is a reliability and availability fix, not a direct theft-of-funds bug, but deadlocks in a hardware wallet's USB path can affect device responsiveness and error recovery.

AI review queuedtext fixesby Daniel Newton · 1f9b4b86 · Oct 22, 2025 · 2 filesMessage 0 · OpaqueInformational 15Details
Commit message · Daniel Newton

text fixes

0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit only fixes a few on-screen text labels and messages. It changes phrases like 'Setup Jade' to 'Set Up Jade', 'temporarily login using' to 'temporarily log in with', and 'Logout' to 'Log Out'. There are no code logic, security, or functional changes.

Lower-priorityusb: only set the usb connect activity when it is initially createdby Jon Griffiths · dfcb0fb1 · Oct 21, 2025 · 1 fileMessage 65 · AdequateInformational 16Details
Commit message · Jon Griffiths

usb: only set the usb connect activity when it is initially created

its already re-set for us after help is shown.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 16/100

This is a small user-interface fix in Blockstream Jade's USB storage mode. It moves one line of code so that the 'please plug in USB device' screen is only set as the current screen when it is first created, not repeatedly every time the code checks for events. The change appears to prevent the screen from being reset/redrawn unnecessarily after help text is shown. There is no direct evidence in the commit that this is a security fix.

Lower-priorityUpdate readme to show all dependencies needed to run test_jade.pyby Daniel Newton · 24deb001 · Oct 20, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Daniel Newton

Update readme to show all dependencies needed to run test_jade.py

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates the README.md file to add clearer instructions for installing test dependencies (virtualenv, bluez-tools, and an additional requirements file). It is purely a documentation change with no code modifications.

Lower-priorityusb: increase timeout to detect slower usb storage devicesby Daniel Newton · 1ff0e587 · Oct 15, 2025 · 1 fileMessage 50 · ThinInformational 15Details
Commit message · Daniel Newton

usb: increase timeout to detect slower usb storage devices

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit simply waits longer (about 3 seconds instead of 1 second) before showing a 'please insert USB device' prompt when a user plugs in a slow USB storage device. It is a usability fix, not a security fix.

Lower-priorityusb: remove need to unplug usb storage device to finish unloading usb libsby Daniel Newton · e48b0146 · Oct 15, 2025 · 1 fileMessage 50 · ThinInformational 16Details
Commit message · Daniel Newton

usb: remove need to unplug usb storage device to finish unloading usb libs

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 16/100

This commit fixes a small user-experience bug in how Blockstream Jade's USB storage feature shuts down. Previously, the device had to wait for the user to physically unplug the USB storage device before it could finish turning off its USB libraries. The change now actively cuts power to the connected USB device so the software can detect it as 'unplugged' and complete cleanup automatically. There is no direct security vulnerability here; it is a robustness/usability improvement.

Lower-priorityusb: dont create background task for msc_host_installby Daniel Newton · ea5b2fd7 · Oct 15, 2025 · 1 fileMessage 50 · ThinInformational 20Details
Commit message · Daniel Newton

usb: dont create background task for msc_host_install

Eliminates the "Processing..." freeze.

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 20/100

This commit changes how Blockstream Jade handles USB mass-storage events. Previously, a separate background task was created to process USB events, which caused a visible "Processing..." freeze on the device. The fix moves event handling into the existing task and makes it run in smaller, more frequent chunks. There is no direct evidence in the commit that this fixes a security vulnerability; it appears to be a user-experience and stability improvement.

Lower-priorityusb: add visual logging system to usb storage code for debuggingby Daniel Newton · 9e7f4ba3 · Oct 15, 2025 · 2 filesMessage 50 · ThinInformational 15Details
Commit message · Daniel Newton

usb: add visual logging system to usb storage code for debugging

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit adds extra on-screen logging and debugging messages for the USB storage feature. It does not change how the device protects keys or data, and it does not fix or introduce a security vulnerability. The new logging is disabled by default.

Security candidatebuild: further verify signature digests in signed v2 binariesby Jon Griffiths · 690c8b1c · Oct 14, 2025 · 1 fileMessage 77 · AdequateLow 42Details
Commit message · Jon Griffiths

build: further verify signature digests in signed v2 binaries

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Mentions testing or verification✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
signing boundary
AI analysis · Low 42/100

This change tightens a build-time signing script for Blockstream Jade hardware wallet firmware. It adds checks that the public-key digests recorded inside a signed binary match the digests of the keys that were actually used to sign it, and makes the script stop on any error. The patch is defensive: it reduces the chance that a corrupted or wrongly-signed firmware image is produced during release, but it does not by itself fix a vulnerability in already-shipped devices.

Security candidatebuild: minor error handling tweak for v2 pem signingby Jon Griffiths · 1d797de3 · Oct 12, 2025 · 1 fileMessage 62 · AdequateInformational 15Details
Commit message · Jon Griffiths

build: minor error handling tweak for v2 pem signing

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Informational 15/100

This is a tiny build-script change that replaces a silent failure with a clear error message when required signing key files are missing. It does not change what the script does, only what it prints when it fails. There is no security vulnerability here.

AI review queuedgui: include semphr.hby Jon Griffiths · 6de067ff · Oct 9, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Jon Griffiths

gui: include semphr.h

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit adds a single missing header file include for FreeRTOS semaphore functions in the GUI source file. It is a straightforward build fix with no visible security relevance in the diff or commit message.

Lower-prioritycbor: avoid passing null to memcpy with len 0 in tinycborby Jon Griffiths · 22af7035 · Oct 8, 2025 · 1 fileMessage 50 · ThinLow 31Details
Commit message · Jon Griffiths

cbor: avoid passing null to memcpy with len 0 in tinycbor

Found by asan.

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Low 31/100

This commit fixes a low-level memory-handling issue in Blockstream Jade's CBOR encoding helper. When adding an empty byte string to a data map, the code previously could pass a null pointer to a memory-copy routine. The fix substitutes a dummy non-null pointer when the data length is zero, avoiding undefined behavior detected by AddressSanitizer. It is a defensive hardening change rather than a confirmed exploitable vulnerability.

Security candidateotp: avoid signed left shiftby Jon Griffiths · 7c8fd718 · Oct 8, 2025 · 1 fileMessage 45 · ThinLow 32Details
Commit message · Jon Griffiths

otp: avoid signed left shift

Found by ubsan.

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
authentication path
AI analysis · Low 32/100

This commit fixes a subtle programming bug in the code that decodes base32 one-time-password (OTP) secrets. The bug involves shifting a signed integer left, which is undefined behavior in C and was caught by an undefined-behavior sanitizer. The fix changes the variable from a signed integer to an unsigned integer so the shift is well-defined. The practical security impact is likely low, but undefined behavior in cryptographic/OTP handling is generally undesirable because compilers may optimize it unpredictably.

Lower-prioritychange QR scan box to use the largest centered square of the camera image (minus a small margin)by Daniel Newton · 719fa40c · Oct 7, 2025 · 3 filesMessage 50 · ThinInformational 13Details
Commit message · Daniel Newton

change QR scan box to use the largest centered square of the camera image (minus a small margin)

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 13/100

This commit changes how the QR code scanning box is sized on the Blockstream Jade hardware wallet. Previously, the scan area was based on how much of the camera image was shown on the device's screen plus a small extra margin. Now it uses the largest centered square that fits in the full camera image, minus a small margin. This is a UI/behavior adjustment, not a security fix.

Security candidateUpdate python/pinserver libwally to 1.5.1by Jon Griffiths · 5e5df8c9 · Sep 29, 2025 · 2 filesMessage 45 · ThinLow 25Details
Commit message · Jon Griffiths

Update python/pinserver libwally to 1.5.1

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
authentication path
AI analysis · Low 25/100

This commit simply updates a dependency called 'wallycore' used by the Blockstream Jade PIN server from version 1.4.0 to 1.5.1. It changes the version number and the list of cryptographic fingerprints (hashes) that verify the downloaded package. The commit message does not say why the update was made or whether it fixes any security issue. Without knowing what changed between 1.4.0 and 1.5.1, we cannot tell if this is a routine maintenance update or a security fix.

Security candidatepsbt: allow pset signing from usb/qrcodesby Jon Griffiths · 4fc854e9 · Sep 29, 2025 · 3 filesMessage 45 · ThinLow 30Details
Commit message · Jon Griffiths

psbt: allow pset signing from usb/qrcodes

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Low 30/100

This commit lets a Blockstream Jade hardware wallet automatically detect whether a transaction file is for Bitcoin or the Liquid sidechain (including test networks) when signing via QR codes or USB. Before, the wallet always assumed Bitcoin, which could cause it to fail or behave incorrectly when signing a Liquid transaction through those offline methods. The change itself is a feature fix and does not appear to introduce a security vulnerability.

Lower-priorityqr: Show brightness controls when displaying animated qrcodesby Jon Griffiths · b9cc4c24 · Sep 29, 2025 · 4 filesMessage 65 · AdequateInformational 15Details
Commit message · Jon Griffiths

qr: Show brightness controls when displaying animated qrcodes

Simplify the associated code since we no longer display a help button
outside of the options section.

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit is a straightforward user-interface change for Blockstream Jade hardware wallets. It replaces a 'help' button with a 'brightness' button on QR code display screens and removes the now-unused help button code. There is no security-relevant change visible in the diff.

Lower-priorityqr: display the parent activity help when showing qr display optionsby Jon Griffiths · 7cb95d4a · Sep 29, 2025 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · Jon Griffiths

qr: display the parent activity help when showing qr display options

Previously this would show help for blkstrm.com/scanjade - which is not
useful when the user is displaying (and not scanning) a qrcode.

Instead, show the help associated with the parent activity where the
options button was selected from.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit fixes a user-interface help link in a QR code display menu. Previously, pressing the help button while viewing options for a displayed QR code would open a help page about scanning QR codes (blkstrm.com/scanjade), which is the opposite of what the user is doing. Now it opens the help page for the screen the user came from. There is no security issue here.

Security candidatebuild: fail if any step of signing v2 firmwares failsby Jon Griffiths · edc17778 · Sep 29, 2025 · 1 fileMessage 62 · AdequateLow 34Details
Commit message · Jon Griffiths

build: fail if any step of signing v2 firmwares fails

62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing boundarysigning or wallet path
AI analysis · Low 34/100

This commit hardens the firmware signing script used when releasing Blockstream Jade hardware wallet firmware. It adds `set -e` so the script stops immediately if any command fails, and explicitly checks that the public and private key files exist before proceeding. Without these changes, the script could silently continue past errors (for example, if a key file was missing or a signing command failed), potentially producing or releasing firmware that was not properly signed. Proper signing is important because users rely on firmware signatures to verify that the software running on their device is authentic and has not been tampered with.