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.
This commit simply adds a new person's PGP public-key fingerprint to the list of trusted release signers in two documentation files. It does not change any code, fix any bug, or alter any security behavior of the software itself.
This is a tiny internal fix to make a bookkeeping migration produce stable event ordering. It changes the timestamp used when creating historical 'deposit' records during a one-time database migration, so the records sort consistently with…
This commit is a documentation-only cleanup of the JSON-RPC command help files. It corrects which numeric error codes are listed for each command so the published schemas match what the software actually returns. No program logic, validati…
Documentation-only change with no executable code modificationsCorrects RPC schema error-code metadata to match actual handler behaviorNo change to input parsing, authorization, cryptography, or network behavior
This commit fixes a bug where a setting that controls whether unexpected transaction signatures are allowed was not initialized when the channel daemon starts. If a peer sent such signatures before the channel was fully ready, the program …
use of uninitialized variableundefined behavior (invalid bool load)network-triggered code path
This commit fixes the project's internal nightly code-coverage CI workflow. It changes how test coverage files are collected, ensures the same LLVM compiler version is used to generate and merge coverage data, and uploads a Codecov-compati…
This commit is a large cleanup of Core Lightning's API schemas, generated RPC bindings, and related plugin code. The stated goal is to make the documented 'required' fields match what the C code actually always produces or expects. In prac…
Large schema-only change with no accompanying security advisory or CVEOne semantic change to plugin hook response: invoice_payment hook can now reject with only failure_message and no resultMany fields change from optional to required in public RPC/protobuf interfaces
This commit only updates documentation. It adds error code 313 to the documented error lists for several Core Lightning commands (fundpsbt, utxopsbt, txprepare, multiwithdraw, and upgradewallet). The error code already existed in the code …
This commit fixes a release-script check that verifies the cryptographic signature on a file of checksums. Previously, the script only told GPG to verify the signature file itself. If someone replaced that signature file with an inline-sig…
Incorrect cryptographic verification logic in release toolingPotential false-positive signature verification with inline-signed .asc substitutionRelease-integrity hardening
This commit updates Core Lightning's release documentation to tell users and release managers to run gpg --verify with both the signature file and the manifest file named explicitly. The old one-argument form can silently succeed even if t…
Verification bypass risk in release artifact validationgpg --verify single-argument form can exit 0 without reading the intended manifestDocumentation-only hardening of release process
This commit is a routine update to the Rust dependency lock file (Cargo.lock), bumping many third-party libraries to newer patch or minor versions. The commit message gives no security reason for the update, and no verified references link…
Routine dependency refresh with no stated security rationaleUpdates to security-sensitive transitive crates (rustls, hyper, h2, tokio, webpki-roots) but no evidence these versions fix known vulnerabilitiesNo source-code changes or patch-specific fixes visible in the diff
This commit fixes a stack-overflow risk in Core Lightning's JSON parser. Before the fix, an attacker could send a valid JSON-RPC message containing thousands of nested brackets or braces. The parser's own helper functions used recursion fo…
Stack-overflow via deeply nested JSONRecursive JSON traversal without depth boundDenial-of-service vector in JSON-RPC input parsing
This commit only fixes typos and comment style. It changes two C-style comments from // to /* */ and corrects a grammar error in a documentation comment ('element' to 'elements'). There are no code behavior changes, no bug fixes, and no se…
This change fixes a test-infrastructure bug in Core Lightning's Python testing helpers. When running tests against a PostgreSQL database, very long test names could be silently shortened by PostgreSQL, causing different test runs or nodes …
No security-relevant signal: change is in test framework code onlyFixes a test reliability issue, not a runtime vulnerabilityNo input sanitization, authentication, cryptography, or network changes
This fix prevents Core Lightning from trying to use freshly created bitcoins (immature coinbase rewards) as emergency funds for fee-bump transactions. Such a transaction would be invalid under Bitcoin's rules and would be rejected by the n…
This commit fixes a bug in Core Lightning's askrene plugin that could prevent a node from restarting. When a saved routing layer contained a node bias with a description, the plugin accidentally freed the description's memory while using i…
Use-after-free / double-take of a tal-allocated string during plugin startupDenial-of-service-like symptom: lightningd aborts before replying to init, node cannot restartFixes publicly reported issue #9433 by endothermicdev
This commit only fixes a test case so it actually exercises the intended code path. It does not change any production code, so it cannot introduce or fix a real-world security vulnerability by itself. The test change is a reproducer for a …
This commit fixes a bug in Core Lightning's experimental dual-funded channel feature. When another node tried to open a channel, Core Lightning was not checking whether the proposed transaction fees were reasonable. A peer could request a …
Missing input validation on wire-parsed feerate fieldsPeer could induce signing and storage of feerate == 0RBF remote path allowed unbounded upward feerate walks
This commit adds regression tests for three related bugs where wildly wrong Bitcoin transaction feerates could enter Core Lightning. In the worst case, a malicious or broken fee source could make the node think a feerate was zero (due to a…
Integer overflow in feerate conversion (u32 wrap from 0xFFFFFFFF perkb to 0 perkw)Absurd feerate from external fee source bypassing sanity ceilingDatabase-stored out-of-range feerate causing startup abort/crash loop
This update fixes a crash bug in Core Lightning. When the software tried to list details of a channel opening in progress, it could crash if a stored fee rate was extremely large or zero. The crash happened because the code used an interna…
Integer overflow in RBF escalation (u32 * 25 / 24) leading to assertion failureAssertion failure in read-only introspection RPC (listpeerchannels) causing crash-loop at startupDatabase value treated as invariant despite originating from external fee estimator
This commit fixes a bug where Core Lightning nodes could get stuck in a crash loop. If a node had previously stored an extremely high or zero fee rate for an in-progress channel funding operation (a 'splice' or dual-funded channel RBF), a …
Integer overflow in fee-rate calculation (u32 overflow when multiplying by 25/24)Assertion failure leading to daemon crash loop at startupDatabase migration clamps out-of-range stored funding feerates
Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.
Lower-priorityaskrene: make increase_flows() use the raw flows array.by Rusty Russell · 2735673f · Nov 17, 2025 · 1 fileMessage 65 · AdequateInformational 11Details
Commit message · Rusty Russell
askrene: make increase_flows() use the raw flows array.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 11/100
This is a small internal code cleanup in Core Lightning's payment routing plugin (askrene). It removes an unnecessary index array and makes a function work directly on the list of payment flows. There is no indication this fixes a security bug or changes externally observable behavior.
Lower-priorityaskrene: use flows array directly in remove_excess.by Rusty Russell · 565a92e8 · Nov 17, 2025 · 1 fileMessage 73 · AdequateInformational 12Details
Commit message · Rusty Russell
askrene: use flows array directly in remove_excess.
We don't need the indexes array, we can use this directly.
We still set up the indexes array (for now) after we call this.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 12/100
This is a small internal code cleanup in the askrene routing plugin. It changes one function to sort and modify a list of payment flows directly instead of going through a separate index array. The old index array is still created afterward for other code that still needs it. There is no indication this fixes a security bug or changes user-visible behavior.
Lower-priorityaskrene: implement reduce_num_flows in refine, using increase_flows().by Rusty Russell · 8c7ac33f · Nov 17, 2025 · 4 filesMessage 85 · StrongLow 33Details
Commit message · Rusty Russell
askrene: implement reduce_num_flows in refine, using increase_flows().
Now we simply call it at the end. We need to check it hasn't violated fee maxima, but otherwise it's simple.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> Changelog-Fixed: Plugins: `askrene` now handles limits on number of htlcs much more gracefully.
85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Low 33/100
This commit fixes the `askrene` routing plugin in Core Lightning so it respects user limits on how many payment paths (HTLCs) can be used. Previously, when a user set `maxparts` lower than the number of routes the algorithm found, the plugin could fail to find a route or behave poorly. The new code merges excess routes into the largest remaining ones and checks that fees stay within budget. It is a correctness/reliability improvement, not a remote exploit.
Lower-priorityaskrene: clean up renamed functions.by Rusty Russell · a8a6a446 · Nov 17, 2025 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Rusty Russell
askrene: clean up renamed functions.
We added _noidx versions of the sort functions, but now they're the only ones, we can rename them to the old names.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100
This is a simple code cleanup: a developer renamed a few internal helper functions by removing the '_noidx' suffix because the older versions no longer exist. The actual behavior of the code is unchanged. There is no security issue here.
Lower-priorityaskrene: remove indexes from refine_flows except for increase_flows()by Rusty Russell · e3dfc020 · Nov 17, 2025 · 1 fileMessage 73 · AdequateInformational 11Details
Commit message · Rusty Russell
askrene: remove indexes from refine_flows except for increase_flows()
This removes the index array from code after increase_flows()m, so we use the flows array directly.
The next step will be to make increase_flows() use the flows array, and remove the index array indirection entirely.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 11/100
This is a small internal code cleanup in the askrene routing plugin. It removes an unnecessary intermediate index array in the flow-refinement logic, making the code operate directly on the flows array. There is no indication this fixes a security bug or changes externally observable behavior.
Lower-priorityaskrene: neated flow array handling, by freeing flows we discard.by Rusty Russell · 6e4bc1dd · Nov 17, 2025 · 1 fileMessage 73 · AdequateLow 25Details
Commit message · Rusty Russell
askrene: neated flow array handling, by freeing flows we discard.
Pointed out by @Lagrang3; he's right, while it's a temporary leak the way we use flows, it's still a trap.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 25/100
This commit fixes a temporary memory leak in the askrene routing plugin of Core Lightning. When the code discarded unused payment routes (called 'flows'), it removed them from a list but forgot to free the memory they occupied. The commit adds a helper function that both removes and frees these discarded flows. The author notes it is a 'temporary leak' and a 'trap' for future developers, not an active security vulnerability.
Lower-priorityaskrene: make increase_flows function more generic.by Rusty Russell · 330f51aa · Nov 17, 2025 · 1 fileMessage 73 · AdequateInformational 23Details
Commit message · Rusty Russell
askrene: make increase_flows function more generic.
Rewrite it, so it properly takes into account interactions between flows by using reservations.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 23/100
This commit rewrites a routing helper inside Core Lightning's experimental 'askrene' plugin so it correctly accounts for overlapping payment routes when increasing payment amounts. The old code sorted routes and increased each independently, which could ignore shared channel capacity limits. The new code reserves capacity across all routes, picks the route with the most spare capacity, and stops if it cannot meet the target. There is no direct evidence this is a security fix, but incorrect capacity accounting in payment routing can lead to failed routes, stuck payments, or unintended fee/HTLC behavior.
Lower-priorityaskrene: handle maxparts parameter values 1 and 0.by Rusty Russell · 050b149e · Nov 17, 2025 · 1 fileMessage 65 · AdequateLow 28Details
Commit message · Rusty Russell
askrene: handle maxparts parameter values 1 and 0.
For 1, we use single-path. For 0, reject.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Low 28/100
This commit fixes how the askrene routing plugin handles the maxparts parameter. Previously, setting maxparts to 1 did not force single-path routing, and setting it to 0 was accepted even though it makes no sense. Now, maxparts=1 forces single-path routing, and maxparts=0 is rejected with an error. This is a minor correctness and input-validation improvement, not a critical security fix.
Lower-priorityaskrene: remove max_deliverable cache from increase_flows.by Rusty Russell · 31d28bad · Nov 17, 2025 · 1 fileMessage 73 · AdequateInformational 11Details
Commit message · Rusty Russell
askrene: remove max_deliverable cache from increase_flows.
Make it calculate on demand. This will be useful when we call it from elsewhere.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 11/100
This is a small internal code cleanup in Core Lightning's payment routing plugin (askrene). It removes a precomputed cache of maximum deliverable amounts and instead calculates those values on demand. There is no indication this fixes a security bug or changes externally observable behavior.
Lower-priorityaskrene: remove overzealous cache of channel_data.by Rusty Russell · b6c22045 · Nov 17, 2025 · 1 fileMessage 83 · StrongInformational 12Details
Commit message · Rusty Russell
askrene: remove overzealous cache of channel_data.
This is not worth optimizing that I can see. Using a non-debug build I get the following times for tests/test_askrene.py::test_real_data
Before: 143 seconds
After: 141 seconds.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 12/100
This commit removes a small performance cache from the askrene routing plugin. The author found the cache barely sped up tests, so they simplified the code by recalculating channel values on demand instead of storing them. There is no indication this fixes a security bug; it is a routine code cleanup.
Lower-priorityaskrene: fix error path if we fail sanity checks.by Rusty Russell · b5b4dbf2 · Nov 17, 2025 · 1 fileMessage 68 · AdequateLow 31Details
Commit message · Rusty Russell
askrene: fix error path if we fail sanity checks.
We've already freed the working_ctx, and the fail path does that again.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 31/100
This commit fixes a programming mistake in Core Lightning's 'askrene' plugin where an error-handling path tried to free memory that had already been freed. Re-freeing the same memory can crash the process or corrupt memory. The fix replaces the shared 'goto fail' path with direct returns so the double-free cannot happen.
Lower-priorityaskrene: Remove index indirection from squash_flows, simplify sorting.by Rusty Russell · 7c7a4f87 · Nov 17, 2025 · 2 filesMessage 73 · AdequateInformational 13Details
Commit message · Rusty Russell
askrene: Remove index indirection from squash_flows, simplify sorting.
We don't need to convert to strings, we can compare directly. This removes the final use of the index arrays.
This of course changes the order of returned routes, which alters test_real_biases, since that biases against the final channel in the *first* route.
Took me far too long to diagnose that!
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 13/100
This is an internal code cleanup in the askrene routing plugin. It replaces a string-based sorting trick with a direct pointer comparison when merging duplicate payment routes. The only observable effect is that routes come out in a slightly different order, so one test's expected numbers were updated. There is no indication this fixes or introduces a security vulnerability.
Lower-priorityaskrene: remove now-unused bottleneck_idx from flow_max_deliverable.by Rusty Russell · 69adf387 · Nov 17, 2025 · 3 filesMessage 65 · AdequateInformational 15Details
Commit message · Rusty Russell
askrene: remove now-unused bottleneck_idx from flow_max_deliverable.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100
This is a routine code cleanup in Core Lightning's payment routing plugin (askrene). It removes an unused output parameter called bottleneck_idx from an internal helper function and its callers. There is no change to behavior, no bug fix, and no security relevance visible in the commit.
Security candidateaskrene: fix use-after-free if remove_htlc_min_violations fails.by Rusty Russell · e1202021 · Nov 17, 2025 · 1 fileMessage 78 · AdequateLow 41Details
Commit message · Rusty Russell
askrene: fix use-after-free if remove_htlc_min_violations fails.
It can only fail on overflow, but if it did, the fail path frees working_ctx and returns "error_message".
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
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
memory safety
AI analysis · Low 41/100
This commit fixes a rare programming bug in Core Lightning's 'askrene' routing plugin. If a specific internal function failed due to an overflow, the code would free a memory context and then try to use it again on the error-handling path, which is a use-after-free. The patch changes the function to use a longer-lived memory context so the error path is safe. The failure condition is described as only possible on overflow, making real-world exploitation unlikely.
AI review queuedcommon: add amount_msat_deduct / amount_msat_deduct_sub.by Rusty Russell · 35f65c5d · Nov 17, 2025 · 21 filesMessage 63 · AdequateInformational 15Details
I added amount_msat_accumulate for the "a+=b" case, but I was struggling with a name for the subtractive equivalent. After some prompting, ChatGPT suggested deduct.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
This commit is a straightforward code cleanup: it introduces two new helper functions, amount_msat_deduct and amount_msat_deduct_sat, which are shorthand for 'subtract this amount from itself' (like a -= b). It then replaces many existing calls of the longer form amount_msat_sub(&x, x, y) with the shorter amount_msat_deduct(&x, y). There is no change in behavior, no bug fix, and no security relevance in the diff itself.
Lower-prioritylightningd: fix access to bcli plugin once it's freed.by Rusty Russell · 0b8239f3 · Nov 16, 2025 · 1 fileMessage 73 · AdequateLow 34Details
Commit message · Rusty Russell
lightningd: fix access to bcli plugin once it's freed.
``` 2025-11-13T07:47:16.5733646Z Valgrind error file: valgrind-errors.46048 2025-11-13T07:47:16.5733739Z ==46048== Invalid read of size 8 2025-11-13T07:47:16.5733869Z ==46048== at 0x2F2B47: strmap_add_ (strmap.c:69) 2025-11-13T07:47:16.5734014Z ==46048== by 0x212066: plugin_request_send (plugin.c:2509) 2025-11-13T07:47:16.5734160Z ==46048== by 0x18FA54: bitcoin_plugin_send (bitcoind.c:123) 2025-11-13T07:47:16.5734329Z ==46048== by 0x190AA7: bitcoind_getrawblockbyheight_ (bitcoind.c:537) 2025-11-13T07:47:16.5734490Z ==46048== by 0x1917E3: bitcoind_getfilteredblock_ (bitcoind.c:857) 2025-11-13T07:47:16.5734615Z ==46048== by 0x1BE2DC: get_txout (gossip_control.c:106) 2025-11-13T07:47:16.5734750Z ==46048== by 0x1BE88A: gossip_msg (gossip_control.c:212) 2025-11-13T07:47:16.5734860Z ==46048== by 0x21BD60: sd_msg_read (subd.c:560) 2025-11-13T07:47:16.5734962Z ==46048== by 0x2E416D: next_plan (io.c:60) 2025-11-13T07:47:16.5735074Z ==46048== by 0x2E4DC3: do_plan (io.c:422) 2025-11-13T07:47:16.5735175Z ==46048== by 0x2E4E2A: io_ready (io.c:439) 2025-11-13T07:47:16.5735294Z ==46048== by 0x2E727E: io_loop (poll.c:455) 2025-11-13T07:47:16.5735464Z ==46048== Address 0x5ec7518 is 216 bytes inside a block of size 288 free'd 2025-11-13T07:47:16.5735692Z ==46048== at 0x484B27F: free (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so) 2025-11-13T07:47:16.5735798Z ==46048== by 0x2F767C: del_tree (tal.c:456) 2025-11-13T07:47:16.5735902Z ==46048== by 0x2F795D: tal_free (tal.c:532) 2025-11-13T07:47:16.5736018Z ==46048== by 0x20C30D: plugin_kill (plugin.c:468) 2025-11-13T07:47:16.5736147Z ==46048== by 0x20D0EE: plugin_conn_finish (plugin.c:853) 2025-11-13T07:47:16.5736265Z ==46048== by 0x2E6AF6: destroy_conn (poll.c:246) 2025-11-13T07:47:16.5736397Z ==46048== by 0x2E6B1A: destroy_conn_close_fd (poll.c:252) 2025-11-13T07:47:16.5736504Z ==46048== by 0x2F6FC9: notify (tal.c:246) 2025-11-13T07:47:16.5736602Z ==46048== by 0x2F75AA: del_tree (tal.c:437) 2025-11-13T07:47:16.5736707Z ==46048== by 0x2F795D: tal_free (tal.c:532) 2025-11-13T07:47:16.5736812Z ==46048== by 0x2E4FF6: io_close (io.c:496) 2025-11-13T07:47:16.5736920Z ==46048== by 0x2E72A5: io_loop (poll.c:459) ```
Changelog-None: exposed by recent plugin changes. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 34/100
This commit fixes a use-after-free bug: when the bitcoin plugin (bcli) was freed during shutdown, the gossip subsystem could still try to send it a request, reading from freed memory. The patch simply stops new plugin requests once the daemon is shutting down. It is a stability/crash bug rather than an obvious remote exploit, and the commit does not describe it as a security vulnerability.
AI review queuedfuzz-tests: Add coverage increasing inputs to seed corporaby Chandra Pratap · 350090a8 · Nov 16, 2025 · 78 filesMessage 83 · StrongInformational 15Details
Commit message · Chandra Pratap
fuzz-tests: Add coverage increasing inputs to seed corpora
Improvements in the fuzz-testing scheme of `fuzz-initial_channel` led to the discovery of test inputs that result in greater code coverage. Add these inputs to the test's seed corpus.
83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidencesecond-pass: unusually broad change
AI analysis · Informational 15/100
This commit only adds new test input files to a fuzz-testing seed corpus. It does not change any production code, configuration, or runtime behavior. The added files are used only during automated fuzz testing to exercise more code paths. There is no direct security impact on users running Core Lightning.
Lower-priorityfuzz-tests: Add test for untested functionby Chandra Pratap · 84b95297 · Nov 16, 2025 · 1 fileMessage 78 · AdequateInformational 15Details
Commit message · Chandra Pratap
fuzz-tests: Add test for untested function
Currently, `fuzz-initial_channel` doesn't verify the function `channel_update_fundinng()` in its target file, `common/initial_channel.h`.
Add a test for it.
78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100
This commit only adds a new fuzz test for an existing function and makes a small constant-definition cleanup. It does not change any production code, fix a bug, or alter behavior of the Core Lightning node. There is no security issue here.
fuzz-tests: Prevent memory leak in `fuzz-initial_channel`
Changelog-None: The current test can leak memory due to improper cleanup in the case of an early return. Fix it.
87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Explains rationale or failure mode✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100
This commit fixes a small memory leak inside a fuzz test, which is an automated testing helper program, not part of the live Lightning node software. It does not affect real users, wallets, or network operations.
script: Add `mtime` constant for reproducible Fedora tarball
Changelog-Fixed: Fedora releases are also deterministic now.
65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 15/100
This commit changes the release build script so that Fedora tarballs use a fixed timestamp when files are packed. This makes the resulting archive byte-for-byte identical every time it is built, which is a reproducible-build improvement. There is no security vulnerability here.
Lower-prioritygitignore: Add generated files to gitignoreby ShahanaFarooqui · 8fe749e8 · Nov 14, 2025 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · ShahanaFarooqui
gitignore: Add generated files to gitignore
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100
This commit only updates Git ignore lists to exclude automatically generated build artifacts from version control. It does not change any executable code, configuration defaults, or security behavior.
Lower-priorityfuzz: when running as unit tests, allow -v and [corpus...] args.by Rusty Russell · 0d93db77 · Nov 14, 2025 · 1 fileMessage 83 · StrongInformational 15Details
Commit message · Rusty Russell
fuzz: when running as unit tests, allow -v and [corpus...] args.
Hacky parser, not a real one, but this is for devs, so they can clean it up with ccan/opt themselves if the want to be fancy! 🎩
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100
This commit only changes the fuzz testing helper code used by developers. It lets developers pass a -v flag and pick specific test corpus files when running fuzz targets as ordinary unit tests. There is no change to production wallet, networking, or consensus code, and nothing here affects real users or live funds.
AI review queuedfuzz-tests: Add a seed corpus for the new testby Chandra Pratap · 0cabd46c · Nov 14, 2025 · 188 filesMessage 78 · AdequateInformational 15Details
Commit message · Chandra Pratap
fuzz-tests: Add a seed corpus for the new test
Add a minimal input set as a seed corpus for the newly introduced test. This leads to discovery of interesting code paths faster.
78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidencesecond-pass: unusually broad change
AI analysis · Informational 15/100
This commit only adds test data files (a fuzzing seed corpus) for a new fuzz test called fuzz-wireaddr. It does not change any production code, so it cannot introduce a runtime security vulnerability on its own. The corpus contains many malformed and unusual network address strings used to exercise the parser during automated testing.
Lower-priorityfuzz-tests: Add a wire test for wireaddr functionsby Chandra Pratap · a59d21dc · Nov 14, 2025 · 1 fileMessage 91 · StrongInformational 15Details
Commit message · Chandra Pratap
fuzz-tests: Add a wire test for wireaddr functions
Changelog-None: `towire_wireaddr()` and `fromwire_wireaddr()` in `common/wireaddr.h` are responsible for marshalling/unmarshalling BOLT #7 address descriptors.
Since these aren't tested by the existing wire fuzz tests, add a roundtrip test for them. This has the added benefit of testing `parse_wireaddr()` as well.
91/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
fuzzing or regression evidence
AI analysis · Informational 15/100
This commit adds a new automated fuzz test file for network address parsing and serialization functions. It does not change any production code, fix a bug, or alter behavior. It is purely a test addition intended to improve code coverage and catch future bugs.
Lower-prioritycommon: fix bad formatting for DNS-type wireaddr.by Rusty Russell · 103cd1c4 · Nov 14, 2025 · 1 fileMessage 70 · AdequateLow 35Details
Commit message · Rusty Russell
common: fix bad formatting for DNS-type wireaddr.
Found by @Chand-ra fuzzing test!
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
fuzzing or regression evidence
AI analysis · Low 35/100
This commit fixes a formatting bug in Core Lightning where DNS-type network addresses were printed as plain text strings without using their known length. The old code passed a pointer to DNS address bytes directly to a string formatter, which could read past the intended data if the bytes were not null-terminated, potentially leaking nearby memory or crashing the program. The fix prints only the exact number of bytes that make up the DNS address.