Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Parity sent 2 transactions with *almost* the same input #7369

Closed
enriquefynn opened this issue Dec 22, 2017 · 3 comments
Closed

Parity sent 2 transactions with *almost* the same input #7369

enriquefynn opened this issue Dec 22, 2017 · 3 comments
Assignees
Labels
F2-bug 🐞 The client fails to follow expected behavior. P5-sometimesoon 🌲 Issue is worth doing soon.
Milestone

Comments

@enriquefynn
Copy link
Contributor

enriquefynn commented Dec 22, 2017

Before filing a new issue, please provide the following information.

I'm running:

  • Which Parity version?: v0.1.0-unstable-d80dd81d7-20171222
  • Which operating system?: MacOS
  • How installed?: from source
  • Are you fully synchronized?: yes
  • Did you try to restart the node?: no

Unfortunately it would be quite difficult to reproduce this behaviour, but it's really scary that it could happen (that's why it's an unstable version I guess)
I signed one transaction to an address never used before, input only one time password, 2 transactions were created with different nonces with the same inputs.
Since I put the gas price extremely low, I was able to cancel one and make the other pass. But if not, both would have passed.

In etherscan it was said there were a 10s gap between the transactions.
Couldn't reproduce the error :-(

@tomusdrw tomusdrw added M7-ui Z0-unconfirmed 🤔 Issue might be valid, but it’s not yet known. labels Dec 27, 2017
@tomusdrw tomusdrw added this to the 1.9 milestone Dec 27, 2017
@tomusdrw
Copy link
Collaborator

@jacogr Could you please make sure that it's not possible to press Confirm button twice on the signer page?

@5chdn 5chdn added the P5-sometimesoon 🌲 Issue is worth doing soon. label Jan 2, 2018
@jacogr jacogr added this to Next Up in User Interface Jan 3, 2018
@jacogr jacogr moved this from Next Up to In Progress in User Interface Jan 4, 2018
@jacogr jacogr removed the Z0-unconfirmed 🤔 Issue might be valid, but it’s not yet known. label Jan 4, 2018
@jacogr
Copy link
Contributor

jacogr commented Jan 4, 2018

Confirmed. There is a breakage on current master when the actual isSending state is not pulled through into the pending item list.

@5chdn 5chdn added the F2-bug 🐞 The client fails to follow expected behavior. label Jan 4, 2018
@jacogr jacogr moved this from In Progress to In Review in User Interface Jan 4, 2018
@jacogr jacogr self-assigned this Jan 4, 2018
@jacogr jacogr moved this from In Review to 1.9.0 in User Interface Jan 4, 2018
@4budab1
Copy link

4budab1 commented Feb 6, 2018

Happened with me aswell, 1.9.2 Windows version, gladly that sum was low and receiver send it back

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
F2-bug 🐞 The client fails to follow expected behavior. P5-sometimesoon 🌲 Issue is worth doing soon.
Projects
No open projects
Development

No branches or pull requests

5 participants