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.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
51/100 average clarity
20Strong · 80–100
123Adequate · 60–79
534Thin · 40–59
66Opaque · 0–39
14security candidates with opaque commit messaging
This Monero wallet update fixes a bookkeeping bug. When a wallet imported a list of its owned outputs and that list was smaller than a previous import, internal lookup maps (key images and public keys) could still point to entries that no …
Internal index/cache consistency fix in wallet output handlingPrevents out-of-range references after transfer list resizeAdds defensive repair on wallet cache load for unrefreshed wallets
This commit simplifies how Monero's simplewallet decides whether a payment ID is a real encrypted ID or a dummy placeholder. Previously, the wallet checked both the payment ID value and whether any destination was an integrated address, an…
Removal of a CHECK_AND_ASSERT_MES consistency check between destination flags and payment ID valueChange from dual-factor classification (address type + payload) to payload-only classificationPotential for previously rejected transactions to be accepted if the old check was overly strict
This Monero wallet patch cleans up internal lookup tables (key-image and public-key indexes) when the list of owned transaction outputs is shrunk, for example during an output import. Without the cleanup, those indexes could point to entri…
Out-of-bounds index retained in wallet lookup maps after container shrinkPotential wallet crash or incorrect spend selection due to stale key-image / public-key mappingRepair-on-load for legacy wallet caches that predate the fix
This small change removes a consistency check in Monero's command-line wallet when loading a saved transaction. Previously, the wallet verified that a 'dummy' payment ID label matched a special zero-value payment ID. Now it labels the paym…
Removal of a CHECK_AND_ASSERT_MES consistency checkRemoval of integrated-address validation for payment ID classificationChange in UI/labeling logic for loaded transactions
This commit updates Monero's built-in blockchain checkpoints to match the v0.18.5.3 release. Checkpoints are hard-coded reference points that help nodes quickly verify they are following the correct chain and resist certain attacks. The ch…
Hard-coded blockchain checkpoint data updated to a newer height/hashExpected compiled-in block hashes digest changedNo new code paths, cryptographic changes, or bug fixes visible in the diff
This is a small code fix in Monero's wallet that keeps the recorded breakdown of received amounts consistent when a transaction output is 'burnt' (replaced or spent as part of a transaction the wallet is processing). The change adds a chec…
Defensive consistency check added (THROW_WALLET_EXCEPTION_IF)Fixes internal accounting of received output amountsNo explicit security claim in commit or supplied references
This patch fixes a spot in the Monero wallet where an untrusted remote server (daemon) could supply a misleading 'status' field. Previously, the wallet used that raw status directly in its error handling, which could potentially make a mal…
Untrusted input from remote daemon used in error-handling pathMissing trust check on daemon-reported RPC statusSingle-call-site hardening patch
This commit adds two new read-only options to the wallet's remote procedure call (RPC) interface so users can request their public view key and public spend key. Public keys are meant to be shared openly and are not secrets, so exposing th…
This commit only updates a submodule pointer for the external polyseed library from one commit hash to another. The actual code changes inside the submodule are not shown in the diff, and no public references were supplied. There is no vis…
This patch fixes a size-limit accounting bug in Monero's built-in HTTP server. When a client sends multiple HTTP requests back-to-back on the same connection (pipelining), leftover buffered data from the next request was not being counted …
Request size limit bypass via pipelined HTTP cachingDenial-of-service / memory pressure potential from oversized requestsFix in low-level network protocol handler
This patch fixes a privacy leak in Monero's wallet RPC server. Previously, the --no-dns flag was ignored when no wallet was loaded, so the validate_address RPC call could still perform a DNS lookup (OpenAlias) even though the user had expl…
Privacy leak: RPC ignored --no-dns when no wallet loadedDNS lookup performed despite explicit user opt-outOpenAlias resolution could disclose queried addresses/aliases to DNS resolvers
This commit adds cleanup of three additional memory buffers in Monero's seed-phrase key-stretching code. Before the change, leftover copies of intermediate secrets could remain in stack memory after the function finished. An attacker who c…
Sensitive intermediate key material left in stack memory after function returnUse of sodium_memzero to clear cryptographic buffersPBKDF2 implementation handling mnemonic-derived secrets
This update improves the Monero wallet's command that connects to a network node (daemon). It lets users supply a username/password and a proxy address when switching daemons, and it replaces an older one-step connection method with a newe…
Added mutex lock in wallet2::set_proxy to protect concurrent access to proxy and HTTP client stateset_daemon now passes RPC login credentials and proxy settings through the proper wallet2::set_daemon APITrust heuristic changed to only auto-trust local daemons when no proxy is in use
This change alters how Monero creates a view-only wallet copy. Previously, the code exported outputs using a method that could strip some metadata. Now it copies the full internal transfer records directly, then wipes and clears the multi-…
Sensitive field sanitization before export (memwipe + clear of m_multisig_k)Change in data export path for view-only wallet creationPreservation of 'complete output metadata' implying previous path was incomplete
This Monero update hardens how public keys and transaction pubkeys are handled. It moves a low-level 'torsion clearing' routine into the core crypto library, adds checks that wallet/destination addresses are valid points on the main subgro…
Adds main-subgroup membership validation for public address keys (spend/view)Normalizes transaction public keys before use in payment-ID decryption and tx proofsMoves torsion-clearing primitive into core crypto layer to ensure consistent behavior
This Monero update fixes a networking bug where the server could accidentally block all of its worker threads while waiting for slow clients to accept data. If all workers became stuck this way, the node could stop processing any network t…
Removal of blocking condition-variable wait in network send pathFail-fast on send-queue overflow instead of parking worker threadsHTTP handler now propagates send failures and enters error state
This Monero wallet patch adds stronger safety checks when a wallet prepares, signs, or loads multi-step transactions (unsigned transactions, multisig transactions, and cold-device transactions). It verifies that money going into the transa…
Adds duplicate-input detection across transaction setsAdds destination address type consistency checksAdds uint64 overflow guard for summed input amounts
This Monero update makes the network layer clean up leftover block download records when a peer connection fails or is rejected. Previously, rejected or disconnected peers could leave stale block spans in a queue, which might cause the nod…
Denial-of-service resistance: stale block spans from malicious or faulty peers could prevent a node from obtaining valid blocksState cleanup on peer disconnection/rejectionNo authentication or memory-safety bug evident in diff
This small patch fixes a bug in the Monero wallet where, if an output had already been scanned once, the wallet would return early without clearing an error flag. In rare cases this could leave a stale 'error' state attached to a transacti…
Stale error-state propagation in wallet scanning logicMissing reset of tx_scan_info.error on cached/short-circuit code pathPotential for incorrect received-payment or scan-failure reporting
This change fixes a binary search in the Monero wallet that previously could loop forever or behave incorrectly if something went wrong. The old code used an unbounded 'while (true)' loop and a midpoint calculation that could overflow. The…
Unbounded loop replaced with bounded iterationInteger overflow mitigation in midpoint calculationDefensive error handling added for search failure
depends: explicitely enable cross-compile mode for autoconf
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityFixed dangling iterator in is_remote_host_allowedby SChernykh · f93346ab · May 24, 2026 · 1 fileMessage 68 · AdequateTriage 0Details
Commit message · SChernykh
Fixed dangling iterator in is_remote_host_allowed
- m_blocked_subnets.erase(it) either moves it to the next entry, or sets it to m_blocked_subnets.end() in which case the MCLOG_CYAN will access invalid data
68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Lower-prioritynet: canonicalize Tor and I2P hostnamesby selsta · 438f7f9f · May 22, 2026 · 4 filesMessage 45 · ThinTriage 0Details
Commit message · selsta
net: canonicalize Tor and I2P hostnames
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-prioritydepends: unbound: update to 1.25.1by tobtoht · 029aa2a2 · May 22, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · tobtoht
depends: unbound: update to 1.25.1
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-prioritydepends: libusb: update to 1.0.30by tobtoht · 363dbd3a · May 22, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · tobtoht
depends: libusb: update to 1.0.30
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityci: guix: use mirrorby tobtoht · 9f5abcb3 · May 22, 2026 · 2 filesMessage 40 · ThinTriage 0Details
Commit message · tobtoht
ci: guix: use mirror
40/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope! No meaningful explanatory body
AI review queuedwallet2: guard optimized coinbase refresh against empty voutby selsta · 57f4cda1 · May 22, 2026 · 1 fileMessage 50 · ThinLow 29Details
Commit message · selsta
wallet2: guard optimized coinbase refresh against empty vout
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 29/100
This patch fixes a corner case in Monero's wallet refresh logic. When scanning a coinbase (miner) transaction with no outputs, the wallet could previously call an internal output-generation helper with zero outputs, which may cause an error or undefined behavior during refresh. The fix skips that work when there are no outputs. It is a defensive hardening change rather than a clear exploit path for stealing funds.
Lower-priorityci: guix: update to 1.5.0by tobtoht · 27a99923 · May 22, 2026 · 1 fileMessage 57 · ThinTriage 0Details
Commit message · tobtoht
ci: guix: update to 1.5.0
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Moderate 61/100
This patch fixes the way Monero's network code reads incoming message headers. Previously, the code directly cast raw byte buffers to structured header types, which can crash or behave incorrectly on some processors when the memory address is not properly aligned. The fix copies the bytes into a properly aligned variable first using memcpy. This is a robustness/correctness fix that may prevent denial-of-service crashes from malformed or specially positioned network data, but the commit message does not frame it as a security vulnerability.
Lower-priorityserialization: handle EOF at varint boundaryby selsta · 1bed4e8f · May 21, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · selsta
serialization: handle EOF at varint boundary
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Security candidateAdd wire::json_writer, and use in some of ZMQ-PUB functions.by Lee *!* Clagett · 108244cd · May 21, 2026 · 22 filesMessage 50 · ThinInformational 18Details
Commit message · Lee *!* Clagett
Add wire::json_writer, and use in some of ZMQ-PUB functions.
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 18/100
This commit refactors how some Monero ZeroMQ publication messages are serialized to JSON. It introduces a new internal JSON writer library and switches a few ZMQ-PUB message types over to it. The change is primarily a code cleanup and modernization; it does not appear to fix a known security bug or introduce an obvious new vulnerability. One small improvement is that serialization errors are now checked and logged instead of silently producing malformed output.
Security candidatetx_pool: track sensitive flag per-tx in added_txs_by_idby j-berman · 1aea9cc2 · May 21, 2026 · 2 filesMessage 73 · AdequateModerate 51Details
Commit message · j-berman
tx_pool: track sensitive flag per-tx in added_txs_by_id
Replace the plain time_t map value with an added_tx_info struct that stores both receive_time and a sensitive flag. Thread the flag through all add_tx_to_transient_lists call sites so incremental RPC queries can skip sensitive transactions.
Reported-by: selsta
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
cryptography-sensitive path
AI analysis · Moderate 51/100
This Monero code change fixes how the transaction memory pool tracks 'sensitive' transactions—those that have not yet been publicly broadcast, such as Dandelion++ stem-phase transactions. Previously, incremental RPC queries could leak these sensitive transactions to clients that should not see them. The patch adds a per-transaction 'sensitive' flag so that RPC responses can skip these transactions. This is a privacy/confidentiality fix, not a direct theft-of-funds bug.
✓ Subject identifies a change✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Moderate 60/100
This Monero update fixes a problem where the daemon's 'get blocks' RPC reply could grow too large and exceed network packet limits. The patch makes the mempool (pool of pending transactions) portion of the response respect a size budget, so it won't be added if the blocks part already fills most of the safe packet size. It also adds internal limits on how many transaction details and how much total transaction blob data are returned. The likely goal is to prevent oversized replies that could cause connection failures or denial-of-service issues for wallets and other nodes.
AI review queuedwallet2: clear m_additional_tx_keys during bg syncby selsta · b3cd06df · May 20, 2026 · 1 fileMessage 50 · ThinLow 35Details
Commit message · selsta
wallet2: clear m_additional_tx_keys during bg sync
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 35/100
This commit fixes a cleanup oversight in the Monero wallet. When the wallet clears private user data during background synchronization, it now also clears 'additional transaction keys' (m_additional_tx_keys). Previously, these extra keys were left behind while other similar secrets were wiped. This could mean sensitive cryptographic material persists longer than intended, potentially increasing exposure if an attacker later accesses wallet memory or state. The change is small and defensive, matching how related secrets are already handled.
Lower-priorityci: guix: use https for substitute serverby tobtoht · 7cc182b2 · May 20, 2026 · 1 fileMessage 57 · ThinTriage 0Details
Commit message · tobtoht
ci: guix: use https for substitute server
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedtests: replace monotonicby tobtoht · d1eaf54e · May 20, 2026 · 3 filesMessage 38 · OpaqueInformational 15Details
Commit message · tobtoht
tests: replace monotonic
38/100 · OpaqueMessage clarity
✓ Subject identifies a change✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This is a routine test-code cleanup. The change swaps a third-party Python package called 'monotonic' for Python's built-in 'time.monotonic()' in the test suite. It does not touch the Monero wallet, daemon, consensus, cryptography, or network code, and it does not fix or introduce any security vulnerability.
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 49/100
This commit adds a safety check to the Monero wallet's remote-control interface (RPC server). The check prevents a specific operation—creating a cryptographic 'spend proof'—from running while the wallet is performing a background synchronization. Without this guard, the operation could potentially use incomplete or inconsistent wallet state, which might lead to incorrect proof results or unexpected behavior. The issue was reported by an independent contributor, ro1m.
Lower-prioritymnemonics: use more idiomatic name for russianby jpk68 · 8e0e49b9 · May 20, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · jpk68
mnemonics: use more idiomatic name for russian
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedwallet2: refactor multisig ignore sets logic into separate functionsby koe · d7370721 · May 20, 2026 · 2 filesMessage 50 · ThinInformational 12Details
Commit message · koe
wallet2: refactor multisig ignore sets logic into separate functions
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 12/100
This commit is a code cleanup: it moves existing multisig signer-selection logic out of the middle of a large transaction-building function into two smaller, named helper functions. The actual behavior appears unchanged. There is no indication this fixes a security bug or introduces a vulnerability.
Lower-prioritycmake: add support for sccacheby jpk68 · 1f848bf6 · May 19, 2026 · 6 filesMessage 45 · ThinTriage 0Details
Commit message · jpk68
cmake: add support for sccache
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 19/100
This commit removes an unused RPC command called finalize_multisig from the Monero wallet's remote-control interface. The command previously did nothing except return an error. Removing it is a cleanup change, not a security fix.
Lower-priorityutils: add missing rpc flags to fish autocompleteby selsta · b3938303 · May 19, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · selsta
utils: add missing rpc flags to fish autocomplete
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Security candidatenet_ssl: Use EVP API to create EVP_PKEYs when availableby jeffro256 · aa3abe2c · May 19, 2026 · 2 filesMessage 78 · AdequateInformational 19Details
Commit message · jeffro256
net_ssl: Use EVP API to create EVP_PKEYs when available
The following functions are deprecated in OpenSSL 3.0 and used in net_ssl.cpp: `RSA_free`, `EC_KEY_free`, `RSA_new`, `RSA_generate_key_ex`, `EC_KEY_set_group`, `EC_KEY_generate_key`.
Since EVP_RSA_gen was added in OpenSSL 3.0 and since the aforementioned functions were deprecated in OpenSSL 3.0, I check if the OpenSSL version >= 3.0, and use EVP_RSA_gen instead of everything else up to assigning the private key to the certificate in `create_rsa_ssl_certificate`.
I also deleted `create_ec_ssl_certificate` since it wasn't used.
78/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Names security-relevant behavior explicitly
Why it was queued
secret or key materialsigning boundary
AI analysis · Informational 19/100
This commit updates Monero's SSL certificate generation code to use newer OpenSSL 3.0 APIs instead of older, deprecated ones. It is primarily a compatibility and cleanup change: it replaces low-level RSA key creation with the modern EVP_RSA_gen function when OpenSSL 3.0+ is available, and removes an unused elliptic-curve certificate function. There is no direct security vulnerability being fixed here, but keeping up with current APIs reduces future maintenance risk and avoids deprecation warnings.
Lower-priorityworkflows: fix ccache dir pathby selsta · 841ef743 · May 18, 2026 · 2 filesMessage 45 · ThinTriage 0Details
Commit message · selsta
workflows: fix ccache dir path
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body