diff options
| author | Wei-Ning Huang <w@dexon.org> | 2018-11-19 22:08:53 +0800 |
|---|---|---|
| committer | Wei-Ning Huang <w@dexon.org> | 2018-12-19 20:54:27 +0800 |
| commit | aa94dff6df6d6ebfe59aba8de01190ffdc409053 (patch) | |
| tree | a1ae6cf5aa61a3f3bce1975c2fc764a8c8ae0572 /core/vm | |
| parent | 3314c0573822d5008211e076831258899565ba5d (diff) | |
| download | dexon-aa94dff6df6d6ebfe59aba8de01190ffdc409053.tar dexon-aa94dff6df6d6ebfe59aba8de01190ffdc409053.tar.gz dexon-aa94dff6df6d6ebfe59aba8de01190ffdc409053.tar.bz2 dexon-aa94dff6df6d6ebfe59aba8de01190ffdc409053.tar.lz dexon-aa94dff6df6d6ebfe59aba8de01190ffdc409053.tar.xz dexon-aa94dff6df6d6ebfe59aba8de01190ffdc409053.tar.zst dexon-aa94dff6df6d6ebfe59aba8de01190ffdc409053.zip | |
core: vm: change offset type to int256
Since we use -1 to represent the offset of empty value, we should make
the return type int256 instead of uint256.
Diffstat (limited to 'core/vm')
| -rw-r--r-- | core/vm/governance.go | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/core/vm/governance.go b/core/vm/governance.go index 36b37b53c..f4c2f4d93 100644 --- a/core/vm/governance.go +++ b/core/vm/governance.go @@ -56,7 +56,7 @@ const GovernanceABIJSON = ` "outputs": [ { "name": "", - "type": "uint256" + "type": "int256" } ], "payable": false, @@ -313,7 +313,7 @@ const GovernanceABIJSON = ` "outputs": [ { "name": "", - "type": "uint256" + "type": "int256" } ], "payable": false, @@ -2044,6 +2044,11 @@ func (g *GovernanceContract) delegate(nodeAddr common.Address) ([]byte, error) { caller := g.contract.Caller() value := g.contract.Value() + // Can not delegate if no fund was sent. + if value.Cmp(big.NewInt(0)) == 0 { + return nil, errExecutionReverted + } + // Can not delegate if already delegated. delegatorOffset := g.state.DelegatorsOffset(nodeAddr, caller) if delegatorOffset.Cmp(big.NewInt(0)) >= 0 { |