btccom / btcpool-ABANDONED

backend of pool.btc.com
https://pool.btc.com
MIT License
643 stars 408 forks source link

Move RSK output to be the last output #469

Open ignaciopulicedonatto opened 4 years ago

ignaciopulicedonatto commented 4 years ago

To avoid possible malleability attacks, RSK discards solutions with more than 128 bytes after RSK tag. Therefor, RSK output should be one of the last or the last output in the Coinbase.

RSKd code condition could be found here: https://github.com/rsksmart/rskj/blob/master/rskj-core/src/main/java/co/rsk/validators/ProofOfWorkRule.java#L202