Skip to content
This repository has been archived by the owner on Oct 4, 2019. It is now read-only.

Send both ETH and ETC to an address, wth happen ?. #17

Closed
qhatgena opened this issue Aug 6, 2016 · 5 comments
Closed

Send both ETH and ETC to an address, wth happen ?. #17

qhatgena opened this issue Aug 6, 2016 · 5 comments

Comments

@qhatgena
Copy link

qhatgena commented Aug 6, 2016

System information

Geth version: Ethereum Classic 1.4.10
OS & Version: Linux
Commit hash : 6aaf5f3

Expected behavior

Receive ETC from valid transaction

Actual behaviour

Receive ETC and ETH from valid transaction

Steps to reproduce the behaviour

  1. Create valid raw transaction
  2. Broadcast to network by use web3.eth.sendRawTransaction() method

Backtrace

https://gastracker.io/tx/0xb35afc0c64cd46e32eb25cd45964babedc214f1aa76c61903ce40f2ced04dedb
http://etherscan.io/tx/0xb35afc0c64cd46e32eb25cd45964babedc214f1aa76c61903ce40f2ced04dedb
@yolandaruiz
Copy link

The transaction is indeed valid on both chains. This is the expected behavior. Nodes are interoperables and will relay the transaction if it's valid for whatever chain they are working on. When you start your node you have peers on both networks.

You should split all your ether before considering any outgoing transaction.

@chiro-hiro
Copy link

chiro-hiro commented Aug 6, 2016

I think that is not a good idea to broadcast to all nodes even it's ETH.

@qhatgena
Copy link
Author

qhatgena commented Aug 6, 2016

@yolandaruiz That answer is not acceptable.

@arvicco
Copy link

arvicco commented Aug 6, 2016

@qhatgena This is the way Ethereum Foundation implemented this fork. They have been told before the fork that such "partial network split" will result in replay attacks, creating problems for Ethereum users. They were fully aware, and they didn't care. There are mitigation strategies to deal with it: https://github.com/ethereumclassic/README/issues/3

@qhatgena
Copy link
Author

qhatgena commented Aug 6, 2016

Ok, that isn't ETC fault. It was so confuse.

@qhatgena qhatgena closed this as completed Aug 6, 2016
soc1c added a commit that referenced this issue Aug 7, 2019
* Fixed state transition logic for edge case (#17)

* Update morden bootnodes (#16)

* Updates bootnodes

* what a fantastic test :)

* Core execution refactor, EIP 684, edge case fixes, starting nonce for morden (#19)

* Refactored Call

* Refactored CallCode

* Refactored DelegateCall

* Refactored StaticCall

* Added create before contract collision check

* Refactored Call

* Refactored CallCode

* Refactored DelegateCall

* Refactored StaticCall

* Added create before contract collision check

* Fixed state transition logic for edge case

* Added create collision logic

* Fix for edge case tests

* Removed outdated contract collision tests and changed format of test output to be consistent

* Rename exec function

* utilize StartingNonce from state instead of explicit 0 nonce checks for contract collision and empty objects

* Fixes morden seg fault edge case

* adding starting nonce to created contracts
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants