Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fusion address reuse #2010

Open
markblundeberg opened this issue Oct 1, 2020 · 2 comments
Open

Fusion address reuse #2010

markblundeberg opened this issue Oct 1, 2020 · 2 comments
Assignees

Comments

@markblundeberg
Copy link

Some fusions have been happening with reused output addresses.

Clues:

  • The fusions are happening around the same time (within a day or so) but not exactly at the same time.
  • Not because multiple copies of wallet open.
@markblundeberg
Copy link
Author

I strongly suspect that this can happen when the electron server connection is down but the connection to fusion server is still good:

  • When fusions finish, they unreserve addresses in order to not bloat the reserved-address data structure. The unreserved addresses will then get tried again as candidates, but, normally this is no problem because now the address has an associated transaction so they will be skipped.
  • If there is a connection issue, then the wallet will have a stale view of coins and then addresses will be tried again.

Mitigations:

  • Don't fuse when wallet is not synced (good idea anyway for other reasons!).
  • Be more careful about unreserving.

@markblundeberg markblundeberg self-assigned this Oct 1, 2020
@cculianu
Copy link
Collaborator

cculianu commented Oct 1, 2020

Hmm your theory seems plausible.

markblundeberg added a commit to markblundeberg/Electron-Cash that referenced this issue Oct 24, 2020
This will hopefully address Electron-Cash#2010 (address reuse).
markblundeberg added a commit to markblundeberg/Electron-Cash that referenced this issue Oct 24, 2020
This will hopefully address Electron-Cash#2010 (address reuse).
markblundeberg added a commit to markblundeberg/Electron-Cash that referenced this issue Oct 26, 2020
This will hopefully address Electron-Cash#2010 (address reuse).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants