9 Commits (5175ec62256999de053e0bf2ecfe13ad3c7468c2)

Author	SHA1	Message	Date
MaxMustermann2	500cabfae2	[core] fix: Move existing CXReceipt check to top	3 years ago
MaxMustermann2	ada5699cc4	[core] fix: cross shard precompile gas to 0... ... ...and ensure transfers from the precompile address can be made before executing them. Since we are now blocking multiple `CXReceipts` per transaction explicitly, it doesn't make sense to be wary of contracts subsidizing cross shard transfers for end users via `delegatecall`. It is then safe to remove the gas charge for the precompile. Secondly, ensure that there are enough funds on the precompile via `CanTransfer` as part of the sanity check.	3 years ago
MaxMustermann2	c9768c9d4e	[core] fix: block multiple cx xfers per tx	3 years ago
MaxMustermann2	961e2b29e4	[core] fix: Use contract object methods ... Instead of hardcoding precompile address and cascading `msg.Value` from evm to the precompile, use methods like `Address()` and `Value()`.	3 years ago
MaxMustermann2	75017bfbad	[core] feat: Add cross shard transfer precompile ... ...for native tokens only, and not for smart contracts (which will be added later). Resolves #4132 and requires a hard fork, currently scheduled at EpochTBD on main and test nets, but epoch 1 (after AcceptsCrossTx) on other nets. The precompile is at address 249 and the method has the signature `crossShardTransfer(uint256 amount, address to, uint32 toShardID)`. It requires a transfer of the `amount` to the precompile address first, in the form of `msg.value`, and cascades a cross shard receipt to the network when executed. At this stage, it is blocked for use by smart contracts by checking that there is no code at the address, and that the address is not a validator. Documentation for Python integration tests to follow.	3 years ago
lutty	219b703714	fix goimports	3 years ago
Lutty	d9e933c7f3	support rosetta tracer	3 years ago
Rongjian Lan	b4b316197a	disable migrate (#4035) ... * disable migrate * comment out more migrate code * fix tests * fix test * fix tests Co-authored-by: Rongjian Lan <rongjianlan@Rongjians-MacBook-Pro-2.local>	3 years ago
Max	d500c4ede6	Resolve harmony-one/bounties#77: Staking precompiles (#3906) ... * Resolve harmony-one/bounties#77: Staking precompiles Create write capable precompiles that can perform staking transactions Add hard fork logic (EpochTBD) for these precompiles Tests for new code with at least 80% unit test coverage Staking library + tests in MaxMustermann2/harmony-staking-precompiles * Fix small typo in comment * Run goimports on files to fix Travis * Do not activate staking precompile on shard 0 * Cascade readOnly to WriteCapableContract * No overlap in readOnly + writeCapable precompiles * Use function selector instead of directive From Solidity, use abi.encodeWithSelector and match it against the exact ABI of the functions. This allows us to remove the need for a directive (32) being encoded, and thus saves 28 bytes of data. * Do not allow contracts to become validators As discussed with Jacky on #3906 * Merge harmony-one/harmony/main properly this time * Run goimports * Update gas calculation for staking precompile Please see comment in core/vm/contracts_write.go RequiredGas * Do not allow contract to become validator (2/2) * Cache StakeMsgs from precompiled transactions Add the StakeMsgs to ProcessorResult and cascade them in insertChain * Remove ContractCode fields from validators Since smart contracts can no longer beecome validators, this field is superfluous. Remove it from the Wrapper structure, and do not assign it a value when creating a validator. Build and goimports checked * Update comments in response to feedback (1) Comments to start with function names (2) Comments for public variables (3) Comment to match function name RunPrecompiledContract (4) Clarify that CreateValidatorFunc + EditValidatorFunc are still used * Fix Travis build by reverting rosetta change * Add revert capability to 3 staking tx types - Delegate - Undelegate - CollectRewards * Fix build: Update evm_test for ValidatorWrapper * Merge main into harmony-staking-precompiles * Add gas for precompile calls and allow EOA usage - Each time the precompile is called, charge the base gas fee plus data cost (if data can be parsed successfully). A gas fee is added to prevent benevolent contract deployers from subsidizing the staking transactions for EOAs through repeated assembly `delegatecall`. - Allow EOAs to use the staking precompile directly. Some changes to the Solidity library are associated with this change. - Remove bytes from parsing address, since the ABI unpacks it into an address format correctly. - Add or update tests. Test coverage report to be attached to the PR shortly. * Run goimports * Check read only and write capable for overlap * Handle precompile stakeMsgs for block proposer The staking precompile generates staking messages which are cascaded to the block via the EVM in `state_processor.go`. This change cascades them in `worker.go` to allow block proposers and block verifiers to keep the same state. * Run goimports for cf2dfac4081444e36a120c9432f4e.. * Update staking precompile epoch to 2 for localnet Bring it in line with staking epoch. Change effects all configurations except mainnet and testnet. `goimports` included. * Add read only precompile to fetch the epoch num * Move epoch precompile to 250 * precompiles: left pad the returned epoch number * chainConfig: check epochs for precompiles panic if staking precompile epoch < pre staking epoch * Add staking migration precompile - Lives at address 251 - Migrates delegations + pending undelegations from address A to B - Useful if address A is hacked - Charges gas of 21k + cost of bytes for two addresses - Does not remove existing delegations, just sets them to zero. Replicates current undelegate setup - Unit tests and `goimports` included. Integration test following shortly in MaxMustermann2/harmony-staking-precompiles * Migration precompile: merge into staking Merge the two precompiles into one, add gas calculation for migration precompile. Move epoch precompile to 251 as a result. When migrating, add undelegations to `To`'s existing undelegations, if any match the epoch. * Add migration gas test, remove panic, add check In response to review comments, add tests for migration gas wherein there are 0/1/2 delegations to migrate. Add the index out of bound check to migration gas calculator and remove panics. Lastly, re-sort migrated undelegations if no existing undelegation in the same epoch was found on `To`. * Move undelegations sorting to end of loop	3 years ago