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
Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.
Security candidateBlockchain: fix data race in get_dynamic_base_fee_estimateby Masamune · 95b207f0 · Jun 12, 2026 · 1 fileMessage 50 · ThinLow 46Details
Commit message · Masamune
Blockchain: fix data race in get_dynamic_base_fee_estimate
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 · Low 46/100
This commit adds a missing lock around a function that estimates transaction fees. Without the lock, multiple threads could read blockchain data while it is being changed, leading to inconsistent or incorrect fee estimates. The fix prevents this 'data race' by ensuring only one thread accesses the relevant data at a time.
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Low 26/100
This commit changes how Monero's daemon returns blocks when a remote caller asks for pruned (trimmed-down) block data. Previously, pruned responses omitted a hash for the prunable part of each transaction. Now the daemon fetches and returns that prunable hash alongside the pruned transaction blob. This is a protocol/data-integrity improvement, not an obvious remote exploit, but it touches the RPC layer and changes the data structure passed between the database, core, and RPC server.
AI review queuedsimplewallet: fix some error messagesby jpk68 · 615d2004 · Jun 10, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit only changes the wording of four user-facing error messages in Monero's command-line wallet. It corrects grammar and fixes a swapped description of multisig threshold parameters. There is no change to program logic, validation, cryptography, or network behavior, so it has no security impact.
AI review queuedfix signed unsigned comparisonby SNeedlewoods · f1ea6167 · Jun 10, 2026 · 1 fileMessage 35 · OpaqueLow 26Details
Commit message · SNeedlewoods
fix signed unsigned comparison
35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Low 26/100
This commit fixes a comparison between a signed and unsigned number in Monero's wallet code that estimates the current blockchain height. Before the fix, mixing signed and unsigned values could lead to incorrect behavior when calculating block height near a future network fork. The patch makes the calculation use only unsigned numbers and keeps the safety check that returns an error if the estimate would go negative. This is a correctness and robustness fix rather than a direct theft-of-funds vulnerability, but bad height estimates could affect transaction creation or fee calculations.
AI review queueduse byte-safe isalpha in epee json value parserby alhudz · 776d3557 · Jun 10, 2026 · 3 filesMessage 45 · ThinModerate 51Details
Commit message · alhudz
use byte-safe isalpha in epee json value parser
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Moderate 51/100
This commit fixes a bug in Monero's JSON parser where it used the standard C 'isalpha' function to decide if a value looks like a keyword such as 'true', 'false', or 'null'. On some systems, passing a negative number (which can happen when the input contains high-byte non-ASCII characters like certain UTF-8 bytes) to the standard 'isalpha' can cause undefined behavior or be misclassified as alphabetic. The patch replaces the unsafe call with a custom byte-safe version that only looks at the raw byte value. A new test confirms that a value starting with a non-ASCII byte is now rejected instead of being treated as a keyword.
AI review queuedwallet2: store multisig nonce erasure before returning signed txsetby selsta · 74f0917d · Jun 9, 2026 · 2 filesMessage 50 · ThinModerate 65Details
Commit message · selsta
wallet2: store multisig nonce erasure before returning signed txset
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: broader security terminologysecond-pass: security-sensitive path
AI analysis · Moderate 65/100
This patch changes how Monero's wallet handles multisig transactions. In a multisig wallet, several people must cooperate to sign a transaction. The patch makes sure that sensitive one-time secret values (called 'nonces') are wiped from the wallet's memory and saved to disk *before* the partially-signed transaction file is handed back to the user. Previously, the wallet could return the signed transaction file while still keeping those secret values in memory. If the wallet later crashed or was used again without saving, those secret values might be reused or leaked, which could weaken the security of the multisig scheme and, in the worst case, allow an attacker to recover private key material or forge signatures.
AI review queuedcombine utf-16 surrogate pairs in json string parserby alhudz · db0f7c55 · Jun 9, 2026 · 2 filesMessage 50 · ThinModerate 51Details
Commit message · alhudz
combine utf-16 surrogate pairs in json string parser
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Moderate 51/100
This commit fixes how Monero's JSON parser handles special Unicode escape sequences called UTF-16 surrogate pairs. Previously, the parser treated each half of a surrogate pair as a separate character, which could produce invalid UTF-8 output or fail to decode characters outside the basic multilingual plane correctly. The patch now combines the two halves into the correct single Unicode character and rejects malformed or lone surrogates. This is a correctness fix in JSON string parsing; it does not appear to be a critical security vulnerability on its own, but malformed Unicode handling can sometimes lead to downstream issues in systems that process the parsed strings.
AI review queuedfix typo in hashchain bounds errorby woodser · 06d3258d · Jun 8, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · woodser
fix typo in hashchain bounds error
45/100 · ThinMessage clarity
✓ 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 15/100
This commit fixes a spelling mistake in an error message. The phrase 'bounds of of hashchain' is corrected to 'bounds of hashchain'. There is no change to program logic, behavior, or security.
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 19/100
This commit fixes a bug in Monero's built-in miner where mining thread #0 was accidentally running in a slower, more secure mode meant for non-mining code. The fix makes all mining threads run with the faster miner-optimized mode. It is a performance bug, not a security vulnerability, and it only affects users who actively use the built-in miner.
✓ Subject identifies a change✓ Provides an explanatory body✓ Links an issue, advisory, or supporting reference
Why it was queued
cryptography-sensitive pathseed or entropy pathsigning or wallet pathboot or update pathauthentication pathparser or protocol pathmerge-commit duplicate discount
AI analysis · Informational 15/100
This commit is a massive initial import or merge of the entire Monero repository, adding 1,300 files including source code, build scripts, CI workflows, documentation, tests, and submodules. The only actual code change described in the commit message is a tiny addition of a helper function `sc_1()` in the crypto module. There is no indication this is a security patch or fixes any vulnerability.
Security candidatecrypto: fix strict aliasing for expandedKey in aesb.cby jeffro256 · c1f7f0bd · Jun 7, 2026 · 1 fileMessage 50 · ThinLow 31Details
Commit message · jeffro256
crypto: fix strict aliasing for expandedKey in aesb.c
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 · Low 31/100
This commit fixes a C programming rule violation called 'strict aliasing' in Monero's AES encryption helper code. Previously, the code treated a byte buffer as if it were an array of 32-bit integers by casting the pointer directly. The patch copies the bytes into a properly typed local array first. This is a correctness and portability fix; under aggressive compiler optimization, the old code could theoretically produce wrong encryption results or expose subtle bugs, but there is no direct evidence it was exploitable as a security vulnerability.
AI review queuedwallet: read RPC crypto fields with memcpyby Ap4sh · 08811dba · Jun 6, 2026 · 1 fileMessage 45 · ThinLow 38Details
Commit message · Ap4sh
wallet: read RPC crypto fields with memcpy
45/100 · ThinMessage clarity
✓ 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 38/100
This commit changes how the Monero wallet RPC server reads certain fixed-size cryptographic identifiers (payment IDs and transaction IDs) from user-supplied blobs. It replaces direct pointer casting with explicit memory copying. The main practical effect is avoiding undefined behavior when the input buffer is not properly aligned for the target type, which could in theory cause crashes or misbehavior on strict CPU architectures. There is no direct evidence in the commit that this fixes an exploitable security vulnerability.
Security candidateunit_tests: test every single field of JSON tx reprby jeffro256 · 360db8f6 · Jun 5, 2026 · 1364 filesMessage 60 · AdequateInformational 15Details
Commit message · jeffro256
unit_tests: test every single field of JSON tx repr
Co-authored-by: selsta <selsta@sent.at>
60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
cryptography-sensitive pathseed or entropy pathsigning or wallet pathboot or update pathauthentication pathparser or protocol path
AI analysis · Informational 15/100
This commit is a large repository initialization or merge that adds the entire Monero codebase, including build files, CI workflows, documentation, and tests. The commit title mentions unit tests for JSON transaction representation, but the actual diff shows a massive addition of files rather than a focused code change. There is no indication of a security fix or vulnerability being addressed.
AI review queuedFix DNS resolve UBby Lee *!* Clagett · 4a8c8631 · Jun 5, 2026 · 1 fileMessage 28 · OpaqueModerate 54Details
Commit message · Lee *!* Clagett
Fix DNS resolve UB
28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Moderate 54/100
This patch fixes a threading bug in Monero's peer-discovery code. The old code launched background threads that wrote into a local variable (`dns_results`) owned by the main function. If the main function finished or the variable went out of scope while a thread was still running, the thread would write to freed memory. That is undefined behavior and could crash the node or, in theory, be abused to corrupt memory. The fix moves the shared data into a heap-allocated structure kept alive with a smart pointer and protects it with a mutex, so threads can finish safely even if the main code no longer waits for them.
AI review queuedread import blob crypto fields with memcpy in wallet2by alhudz · af10a4f6 · Jun 5, 2026 · 1 fileMessage 50 · ThinLow 38Details
Commit message · alhudz
read import blob crypto fields with memcpy in wallet2
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 38/100
This commit changes how Monero's wallet reads cryptographic data from imported files. It replaces direct pointer casts with explicit memory copying (memcpy). This is a defensive coding fix that primarily addresses alignment and strict-aliasing issues, which can cause crashes or undefined behavior on some platforms. It is not a clear-cut remote exploit fix, but it removes a class of low-level memory-safety risks when loading wallet-related import data.
AI review queuedepee: use correct minor http versionby jpk68 · 49108d7a · Jun 5, 2026 · 1 fileMessage 45 · ThinLow 29Details
Commit message · jpk68
epee: use correct minor http version
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
parser or protocol pathsecond-pass: security-sensitive path
AI analysis · Low 29/100
This is a one-line bug fix in Monero's embedded HTTP protocol handler. The code was accidentally passing the major HTTP version number twice to a parsing function, instead of passing both the major and minor version numbers. The fix makes the parser receive the correct minor version number. This is clearly a bug, but from the diff alone it is hard to tell whether it has any practical security consequence.
Defer loading of private view key from device until first time it is needed. Do not fail if this fails. This has two effects:
1. The prompt to export the view key is only needed once when creating a `cryptonote::account_base` 2. The call to `connect()` doesn't fail if the user decies to not export the viewkey, and thus usage of the device without exporting view keys is possible
This is a small convenience for Ledger users, but will be an even larger convenience for Ledger device testing
78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
privacy or spend-authorization protocol
AI analysis · Informational 18/100
This change makes Monero's Ledger hardware wallet integration ask for the private view key only when it is first needed, rather than immediately when connecting. If the user refuses, the software no longer fails and can continue in a slower mode where the Ledger does more cryptographic work. The change also clears the cached view key from memory when disconnecting. It is a usability improvement, not a fix for an active security flaw.
Fix: check hwdev.generate_output_ephemeral_keys result
If it fails, out_eph_public_key and view_tag remain uninitialized.
77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Moderate 59/100
This commit fixes a bug where Monero's transaction creation code did not check whether a hardware wallet device successfully generated the one-time public key and view tag for a transaction output. If the hardware device call failed, the code would continue using uninitialized cryptographic values, which could lead to creating an invalid or insecure transaction. The fix adds an explicit error check that aborts transaction creation if the key generation fails.
Security candidateRefactored tx verification NIC logicby SChernykh · 196078ea · Jun 4, 2026 · 5 filesMessage 70 · AdequateLow 46Details
Commit message · SChernykh
Refactored tx verification NIC logic
Encapsulated `nic_verified_hf_version` and `txs_by_txid` to make it impossible for callers to skip updating it.
70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
defensive validationcryptography-sensitive pathparser or protocol path
AI analysis · Low 46/100
This commit is a defensive code cleanup in Monero's transaction verification system. It restructures how a temporary list of extra block transactions (the 'pool supplement') is handled so that callers cannot accidentally forget to mark the list as unverified when adding new transactions. Before, outside code directly inserted transactions and separately reset a verification flag, which created a risk that someone could add a transaction without resetting the flag and potentially use a stale 'already verified' status. Now, adding or clearing transactions automatically resets the flag, and only the dedicated verification function can set it to a valid version. There is no direct evidence in the commit of an actual exploit or bug being fixed, but the change removes a fragile pattern that could lead to consensus or security issues.
Blockchain: fix wrong block_weight in handle_get_objects
- When there are missing IDs, blocks.size() != arg.blocks.size(), so arg.blocks can't be indexed by `i` - the indices will be wrong, the wrong weight will be returned to some peer and this peer will ban our node.
Use `bl.second` instead of `arg.blocks[i]`. Also it saves one DB query per returned block.
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 60/100
This commit fixes a bug in Monero's blockchain code where a node could accidentally report the wrong block size (weight) to a peer. The bug happened because the code used a mismatched index when some requested blocks were missing. A peer receiving the wrong weight might ban the node, hurting network connectivity. The fix also removes an unnecessary database lookup.
Security candidatewallet_rpc_server: preserve payment ID when editing address bookby selsta · f095abd9 · Jun 3, 2026 · 2 filesMessage 50 · ThinLow 37Details
Commit message · selsta
wallet_rpc_server: preserve payment ID when editing address book
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
privacy or spend-authorization protocolsigning or wallet path
AI analysis · Low 37/100
This commit fixes a bug in Monero's wallet RPC server where editing an address book entry could accidentally drop or mishandle the payment ID associated with an integrated address. The fix ensures the payment ID flag is preserved correctly, and new tests check that converting between integrated and standard addresses works as expected. In practical terms, this could have caused users to send funds without the intended payment identifier, making transactions harder to track or reconcile.
Security candidatetranslations: don't include locationsby tobtoht · b51a0cb8 · Jun 3, 2026 · 14 filesMessage 45 · ThinInformational 15Details
Commit message · tobtoht
translations: don't include locations
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
boot or update pathtranslation-only discount
AI analysis · Informational 15/100
This commit removes source-code file names and line numbers from Monero's translation files. It is a routine cleanup to make translation updates smaller and less noisy. There is no change to program logic, no bug fix, and no security improvement or regression.
Security candidatetx_pool: fix use-after-free in prune() - txid was a reference to an item which was later deleted in remove_tx_from_transient_lists(), and txid was used after thatby SChernykh · b5f7330e · Jun 3, 2026 · 1 fileMessage 73 · AdequateModerate 59Details
Commit message · SChernykh
tx_pool: fix use-after-free in prune() - txid was a reference to an item which was later deleted in remove_tx_from_transient_lists(), and txid was used after that
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 59/100
This commit fixes a use-after-free bug in Monero's transaction pool (tx_pool). The code was storing a reference to a transaction ID, then deleting the underlying data structure that held that ID, and later still using the now-stale reference. The fix simply copies the ID value instead of holding a reference, so it remains valid after deletion. This is a memory-safety bug that could cause crashes or, in the worst case, be exploited to manipulate transaction processing, though the commit itself does not describe any exploit.
AI review queuedsimplewallet: add wallet-dir flagby jpk68 · ed7d4637 · Jun 3, 2026 · 3 filesMessage 60 · AdequateInformational 20Details
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 20/100
This commit adds a new --wallet-dir command-line option to the Monero command-line wallet (monero-wallet-cli). It lets users specify a single directory where wallet files should be looked for and saved. The change is a convenience/usability feature and does not appear to fix a security bug or introduce a dangerous capability. The code only accepts absolute paths for the directory and ignores the option if an absolute wallet-file path is given, which reduces the chance of tricking the wallet into writing files somewhere unexpected.
AI review queuednet: canonicalize Tor/I2P hosts during deserializationby selsta · 0fc5bb7c · Jun 3, 2026 · 3 filesMessage 50 · ThinLow 48Details
Commit message · selsta
net: canonicalize Tor/I2P hosts during deserialization
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
second-pass: broader security terminology
AI analysis · Low 48/100
This change makes Monero's network address handling for Tor and I2P more consistent by converting hostnames to lowercase when they are read from stored or received data. Before this fix, an address like 'EXAMPLE.onion' written in capital letters would be treated differently from 'example.onion', which could cause mismatches, failed connections, or possibly bypass security checks that rely on exact string comparison. The patch adds a normalization step during deserialization and includes tests to confirm it works.