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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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