Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

reset key index after delete account #1776

Closed
wants to merge 1 commit into from
Closed

reset key index after delete account #1776

wants to merge 1 commit into from

Conversation

qiluge
Copy link

@qiluge qiluge commented Aug 13, 2019

No description provided.

@qiluge
Copy link
Author

qiluge commented Aug 13, 2019

fix #1775

Paladz
Paladz suggested changes Aug 16, 2019
View reviewed changes
Copy link
Contributor

@Paladz Paladz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR will cause a bug in this situation:

  1. account A use index 1, account B use index 2
  2. delete account B delete the index
  3. when user create account C it's using index 1, which means account A == account C

@qiluge
Copy link
Author

qiluge commented Aug 16, 2019

If I use restore-wallet rpc interface to restore my account, will the keyIndex increase?

@Paladz Paladz closed this Sep 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Paladz Paladz Paladz requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@qiluge @Paladz