Skip to content
This repository has been archived by the owner on Jul 20, 2021. It is now read-only.

block height reported by wallet transactions CLI command incorrect #557

Open
GlenDC opened this issue Jul 22, 2019 · 4 comments
Open

block height reported by wallet transactions CLI command incorrect #557

GlenDC opened this issue Jul 22, 2019 · 4 comments

Comments

@GlenDC
Copy link
Contributor

GlenDC commented Jul 22, 2019

A user reported that the block height for certain transactions as returned by the <> wallet transactions std CLI command are incorrect, and in fact about twice the size of the current height of the reported chain height.

@LeeSmet
Copy link
Collaborator

LeeSmet commented Sep 24, 2019

Is there any more info available? There is mention of "certain" transactions - is there an example of such a transactions perhaps?

@LeeSmet
Copy link
Collaborator

LeeSmet commented Sep 24, 2019

Currently unable to reproduce

@GlenDC
Copy link
Contributor Author

GlenDC commented Sep 24, 2019

Nope, I have not more info, have never been able to reproduce it myself, better approach to solve this issue would be by reasoning through the code, as reproducing it doesn't seem to be easy.

@LeeSmet
Copy link
Collaborator

LeeSmet commented Sep 26, 2019

It seems the only place the height is set on the transactions is in the ProcessConsensusChange function in the wallet. This function uses a variable in the wallet to keep track of the current height, which would suggest it is possible for this variable to get out of sync

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants