Page MenuHomePhabricator

wallet: Avoid leaking nLockTime fingerprint when anti-fee-sniping
ClosedPublic

Authored by Fabien on Mar 25 2020, 12:15.

Details

Summary
The wallet sets the locktime to the current height of our active chain.
This is fine, as long as our node is connected to other nodes. However,
when we fall back and get stuck at a particular height (e.g. taking the
wallet offline), the same (potentially unique) locktime is used for all
transactions. This makes it easier for passive observers to cluster
transactions by wallet.

Backport of core PR15039.

Test Plan
ninja check check-functional-extended

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR15039
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9962
Build 17776: Default Diff Build & Tests
Build 17775: arc lint + arc unit

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

This revision is now accepted and ready to land.Mar 25 2020, 18:38