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

Wrong transaction order #146

Closed
awfm9 opened this issue Feb 18, 2013 · 3 comments
Closed

Wrong transaction order #146

awfm9 opened this issue Feb 18, 2013 · 3 comments

Comments

@awfm9
Copy link

awfm9 commented Feb 18, 2013

Hello dear electrum team.

I somehow managed to get two dependent transactions encoded into the same block. After several confirmations of the transactions & synchronization with several electrum servers, I'm still left with the same issue.

Transaction 1, receiving X coins on address (X coins already).
Transaction 2, sending 2X coins from address.

I have no other bitcoins in my wallet.

In the transaction list, transaction 2 is displayed before transaction 1. This leads to a bug where it will display the overall balance as negative (-X) after transaction 2, followed by transaction 1 which brings it back to zero (0).

I guess you could fix it by either having more exact timestamps (not sure how they work), or by simply making sure than if you have transactions that depend on another transaction, to make them follow it instead of being able to show up in front.

Cheers.

@awfm9
Copy link
Author

awfm9 commented Feb 18, 2013

Additionally, while both transactions were still pending, the pruning servers would put one of them into the pruned transactions (behind a LOT of transactions that happened WAY earlier) for some reason, which made the order even more confusing. This was fixed once they got confirmed & timestamped, however.

@ecdsa
Copy link
Member

ecdsa commented Feb 19, 2013

the first issue is known. dependencies detection needs to be implemented.
I was not aware of the second one.

@ecdsa
Copy link
Member

ecdsa commented Mar 23, 2013

this is fixed: transactions are now in the correct order (except when they are still pending)
I cannot reproduce the second issue; it might be a thread synchronisation problem.

@ecdsa ecdsa closed this as completed Mar 23, 2013
rllola pushed a commit to rllola/electrum that referenced this issue Sep 23, 2019
aac2636 Namecoin: Rebrand Kivy About screen (JeremyRand)

Pull request description:

  Fixes namecoin#142
UdjinM6 pushed a commit to UdjinM6/electrum-dash that referenced this issue Oct 1, 2023
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