This commit is contained in:
Martin Holst Swende 2017-12-21 16:18:37 +01:00
parent 73d4a57d47
commit c095c87e11
No known key found for this signature in database
GPG Key ID: 683B438C05A5DDF0
5 changed files with 128 additions and 47 deletions

View File

@ -17,6 +17,7 @@
package abi package abi
import ( import (
"bytes"
"encoding/json" "encoding/json"
"fmt" "fmt"
"io" "io"
@ -133,3 +134,14 @@ func (abi *ABI) UnmarshalJSON(data []byte) error {
return nil return nil
} }
// MethodById looks up a method by the 4-byte id
// returns nil if none found
func (abi *ABI) MethodById(sigdata []byte) *Method {
for _, method := range abi.Methods {
if bytes.Equal(method.Id(), sigdata[:4]) {
return &method
}
}
return nil
}

View File

@ -25,6 +25,8 @@ import (
"strings" "strings"
"testing" "testing"
"reflect"
"github.com/ethereum/go-ethereum/common" "github.com/ethereum/go-ethereum/common"
"github.com/ethereum/go-ethereum/crypto" "github.com/ethereum/go-ethereum/crypto"
) )
@ -74,12 +76,24 @@ func TestReader(t *testing.T) {
} }
// deep equal fails for some reason // deep equal fails for some reason
//t.Skip() for name, expM := range exp.Methods {
// Check with String() instead gotM, exist := abi.Methods[name]
expS := fmt.Sprintf("%v",exp) if !exist {
gotS := fmt.Sprintf("%v", abi) t.Errorf("Missing expected method %v", name)
if expS != gotS { }
t.Errorf("\nGot abi: \n%v\ndoes not match expected \n%v", abi, exp) if !reflect.DeepEqual(gotM, expM) {
t.Errorf("\nGot abi method: \n%v\ndoes not match expected method\n%v", gotM, expM)
}
}
for name, gotM := range abi.Methods {
expM, exist := exp.Methods[name]
if !exist {
t.Errorf("Found extra method %v", name)
}
if !reflect.DeepEqual(gotM, expM) {
t.Errorf("\nGot abi method: \n%v\ndoes not match expected method\n%v", gotM, expM)
}
} }
} }
@ -643,3 +657,42 @@ func TestUnpackEvent(t *testing.T) {
t.Logf("len(data): %d; received event: %+v", len(data), ev) t.Logf("len(data): %d; received event: %+v", len(data), ev)
} }
} }
func TestABI_MethodById(t *testing.T) {
const abiJSON = `[
{"type":"function","name":"receive","constant":false,"inputs":[{"name":"memo","type":"bytes"}],"outputs":[],"payable":true,"stateMutability":"payable"},
{"type":"event","name":"received","anonymous":false,"inputs":[{"indexed":false,"name":"sender","type":"address"},{"indexed":false,"name":"amount","type":"uint256"},{"indexed":false,"name":"memo","type":"bytes"}]},
{"type":"function","name":"fixedArrStr","constant":true,"inputs":[{"name":"str","type":"string"},{"name":"fixedArr","type":"uint256[2]"}]},
{"type":"function","name":"fixedArrBytes","constant":true,"inputs":[{"name":"str","type":"bytes"},{"name":"fixedArr","type":"uint256[2]"}]},
{"type":"function","name":"mixedArrStr","constant":true,"inputs":[{"name":"str","type":"string"},{"name":"fixedArr","type":"uint256[2]"},{"name":"dynArr","type":"uint256[]"}]},
{"type":"function","name":"doubleFixedArrStr","constant":true,"inputs":[{"name":"str","type":"string"},{"name":"fixedArr1","type":"uint256[2]"},{"name":"fixedArr2","type":"uint256[3]"}]},
{"type":"function","name":"multipleMixedArrStr","constant":true,"inputs":[{"name":"str","type":"string"},{"name":"fixedArr1","type":"uint256[2]"},{"name":"dynArr","type":"uint256[]"},{"name":"fixedArr2","type":"uint256[3]"}]},
{"type":"function","name":"balance","constant":true},
{"type":"function","name":"send","constant":false,"inputs":[{"name":"amount","type":"uint256"}]},
{"type":"function","name":"test","constant":false,"inputs":[{"name":"number","type":"uint32"}]},
{"type":"function","name":"string","constant":false,"inputs":[{"name":"inputs","type":"string"}]},
{"type":"function","name":"bool","constant":false,"inputs":[{"name":"inputs","type":"bool"}]},
{"type":"function","name":"address","constant":false,"inputs":[{"name":"inputs","type":"address"}]},
{"type":"function","name":"uint64[2]","constant":false,"inputs":[{"name":"inputs","type":"uint64[2]"}]},
{"type":"function","name":"uint64[]","constant":false,"inputs":[{"name":"inputs","type":"uint64[]"}]},
{"type":"function","name":"foo","constant":false,"inputs":[{"name":"inputs","type":"uint32"}]},
{"type":"function","name":"bar","constant":false,"inputs":[{"name":"inputs","type":"uint32"},{"name":"string","type":"uint16"}]},
{"type":"function","name":"_slice","constant":false,"inputs":[{"name":"inputs","type":"uint32[2]"}]},
{"type":"function","name":"__slice256","constant":false,"inputs":[{"name":"inputs","type":"uint256[2]"}]},
{"type":"function","name":"sliceAddress","constant":false,"inputs":[{"name":"inputs","type":"address[]"}]},
{"type":"function","name":"sliceMultiAddress","constant":false,"inputs":[{"name":"a","type":"address[]"},{"name":"b","type":"address[]"}]}
]
`
abi, err := JSON(strings.NewReader(abiJSON))
if err != nil {
t.Fatal(err)
}
for name, m := range abi.Methods {
a := fmt.Sprintf("%v", m)
b := fmt.Sprintf("%v", abi.MethodById(m.Id()))
if a != b {
t.Errorf("Method %v (id %v) not 'findable' by id in ABI", name, common.ToHex(m.Id()))
}
}
}

View File

@ -34,7 +34,7 @@ type Argument struct {
type Arguments []Argument type Arguments []Argument
// UnmarshalJSON implements json.Unmarshaler interface // UnmarshalJSON implements json.Unmarshaler interface
func (a *Argument) UnmarshalJSON(data []byte) error { func (argument *Argument) UnmarshalJSON(data []byte) error {
var extarg struct { var extarg struct {
Name string Name string
Type string Type string
@ -45,38 +45,43 @@ func (a *Argument) UnmarshalJSON(data []byte) error {
return fmt.Errorf("argument json err: %v", err) return fmt.Errorf("argument json err: %v", err)
} }
a.Type, err = NewType(extarg.Type) argument.Type, err = NewType(extarg.Type)
if err != nil { if err != nil {
return err return err
} }
a.Name = extarg.Name argument.Name = extarg.Name
a.Indexed = extarg.Indexed argument.Indexed = extarg.Indexed
return nil return nil
} }
func countNonIndexedArguments(args []Argument) int { // LengthNonIndexed returns the number of arguments when not counting 'indexed' ones. Only events
// can ever have 'indexed' arguments, it should always be false on arguments for method input/output
func (arguments Arguments) LengthNonIndexed() int {
out := 0 out := 0
for i := range args { for _, arg := range arguments {
if !args[i].Indexed { if !arg.Indexed {
out++ out++
} }
} }
return out return out
} }
func (a *Arguments) isTuple() bool {
return a != nil && len(*a) > 1 // isTuple returns true for non-atomic constructs, like (uint,uint) or uint[]
func (arguments Arguments) isTuple() bool {
return len(arguments) > 1
} }
func (a *Arguments) Unpack(v interface{}, data []byte) error { // Unpack performs the operation hexdata -> Go format
if a.isTuple() { func (arguments Arguments) Unpack(v interface{}, data []byte) error {
return a.unpackTuple(v, data) if arguments.isTuple() {
return arguments.unpackTuple(v, data)
} }
return a.unpackAtomic(v, data) return arguments.unpackAtomic(v, data)
} }
func (a *Arguments) unpackTuple(v interface{}, output []byte) error { func (arguments Arguments) unpackTuple(v interface{}, output []byte) error {
// make sure the passed value is a pointer // make sure the passed value is arguments pointer
valueOf := reflect.ValueOf(v) valueOf := reflect.ValueOf(v)
if reflect.Ptr != valueOf.Kind() { if reflect.Ptr != valueOf.Kind() {
return fmt.Errorf("abi: Unpack(non-pointer %T)", v) return fmt.Errorf("abi: Unpack(non-pointer %T)", v)
@ -87,17 +92,16 @@ func (a *Arguments) unpackTuple(v interface{}, output []byte) error {
typ = value.Type() typ = value.Type()
kind = value.Kind() kind = value.Kind()
) )
/* !TODO add this back
if err := requireUnpackKind(value, typ, kind, (*a), false); err != nil { if err := requireUnpackKind(value, typ, kind, arguments); err != nil {
return err return err
} }
*/
// `i` counts the nonindexed arguments. // `i` counts the nonindexed arguments.
// `j` counts the number of complex types. // `j` counts the number of complex types.
// both `i` and `j` are used to to correctly compute `data` offset. // both `i` and `j` are used to to correctly compute `data` offset.
i, j := -1, 0 i, j := -1, 0
for _, arg := range(*a) { for _, arg := range arguments {
if arg.Indexed { if arg.Indexed {
// can't read, continue // can't read, continue
@ -130,14 +134,16 @@ func (a *Arguments) unpackTuple(v interface{}, output []byte) error {
} }
case reflect.Slice, reflect.Array: case reflect.Slice, reflect.Array:
if value.Len() < i { if value.Len() < i {
return fmt.Errorf("abi: insufficient number of arguments for unpack, want %d, got %d", len(*a), value.Len()) return fmt.Errorf("abi: insufficient number of arguments for unpack, want %d, got %d", len(arguments), value.Len())
} }
v := value.Index(i) v := value.Index(i)
if err := requireAssignable(v, reflectValue); err != nil { if err := requireAssignable(v, reflectValue); err != nil {
return err return err
} }
reflectValue := reflect.ValueOf(marshalledValue)
return set(v.Elem(), reflectValue, arg) if err := set(v.Elem(), reflectValue, arg); err != nil {
return err
}
default: default:
return fmt.Errorf("abi:[2] cannot unmarshal tuple in to %v", typ) return fmt.Errorf("abi:[2] cannot unmarshal tuple in to %v", typ)
} }
@ -145,13 +151,14 @@ func (a *Arguments) unpackTuple(v interface{}, output []byte) error {
return nil return nil
} }
func (a *Arguments) unpackAtomic(v interface{}, output []byte) error { // unpackAtomic unpacks ( hexdata -> go ) a single value
// make sure the passed value is a pointer func (arguments Arguments) unpackAtomic(v interface{}, output []byte) error {
// make sure the passed value is arguments pointer
valueOf := reflect.ValueOf(v) valueOf := reflect.ValueOf(v)
if reflect.Ptr != valueOf.Kind() { if reflect.Ptr != valueOf.Kind() {
return fmt.Errorf("abi: Unpack(non-pointer %T)", v) return fmt.Errorf("abi: Unpack(non-pointer %T)", v)
} }
arg := (*a)[0] arg := arguments[0]
if arg.Indexed { if arg.Indexed {
return fmt.Errorf("abi: attempting to unpack indexed variable into element.") return fmt.Errorf("abi: attempting to unpack indexed variable into element.")
} }
@ -162,19 +169,13 @@ func (a *Arguments) unpackAtomic(v interface{}, output []byte) error {
if err != nil { if err != nil {
return err return err
} }
if err := set(value, reflect.ValueOf(marshalledValue), arg); err != nil { return set(value, reflect.ValueOf(marshalledValue), arg)
return err
}
return nil
} }
func (arguments *Arguments) Pack(args ...interface{}) ([]byte, error) { // Unpack performs the operation Go format -> Hexdata
func (arguments Arguments) Pack(args ...interface{}) ([]byte, error) {
// Make sure arguments match up and pack them // Make sure arguments match up and pack them
if arguments == nil { abiArgs := arguments
return nil, fmt.Errorf("arguments are nil, programmer error!")
}
abiArgs := *arguments
if len(args) != len(abiArgs) { if len(args) != len(abiArgs) {
return nil, fmt.Errorf("argument count mismatch: %d for %d", len(args), len(abiArgs)) return nil, fmt.Errorf("argument count mismatch: %d for %d", len(args), len(abiArgs))
} }

View File

@ -225,7 +225,7 @@ func TestEventTupleUnpack(t *testing.T) {
err := unpackTestEventData(tc.dest, tc.data, tc.jsonLog, assert) err := unpackTestEventData(tc.dest, tc.data, tc.jsonLog, assert)
if tc.error == "" { if tc.error == "" {
assert.Nil(err, "Should be able to unpack event data.") assert.Nil(err, "Should be able to unpack event data.")
assert.Equal(tc.expected, tc.dest) assert.Equal(tc.expected, tc.dest, tc.name)
} else { } else {
assert.EqualError(err, tc.error) assert.EqualError(err, tc.error)
} }
@ -243,11 +243,10 @@ func unpackTestEventData(dest interface{}, hexData string, jsonEvent []byte, ass
} }
/* /*
!TODO enable these when the fix is in. Taken from Taken from
https://github.com/ethereum/go-ethereum/pull/15568 https://github.com/ethereum/go-ethereum/pull/15568
*/ */
/*
type testResult struct { type testResult struct {
Values [2]*big.Int Values [2]*big.Int
Value1 *big.Int Value1 *big.Int
@ -309,10 +308,9 @@ func TestEventIndexedWithArrayUnpack(t *testing.T) {
b.Write(packNum(reflect.ValueOf(32))) b.Write(packNum(reflect.ValueOf(32)))
b.Write(packNum(reflect.ValueOf(len(stringOut)))) b.Write(packNum(reflect.ValueOf(len(stringOut))))
b.Write(common.RightPadBytes([]byte(stringOut), 32)) b.Write(common.RightPadBytes([]byte(stringOut), 32))
fmt.Println(b.Bytes())
var rst testStruct var rst testStruct
require.NoError(t, abi.Unpack(&rst, "test", b.Bytes())) require.NoError(t, abi.Unpack(&rst, "test", b.Bytes()))
require.Equal(t, [2]uint8{0, 0}, rst.Value1) require.Equal(t, [2]uint8{0, 0}, rst.Value1)
require.Equal(t, stringOut, rst.Value2) require.Equal(t, stringOut, rst.Value2)
} }
*/

View File

@ -93,3 +93,20 @@ func requireAssignable(dst, src reflect.Value) error {
} }
return nil return nil
} }
// requireUnpackKind verifies preconditions for unpacking `args` into `kind`
func requireUnpackKind(v reflect.Value, t reflect.Type, k reflect.Kind,
args Arguments) error {
switch k {
case reflect.Struct:
case reflect.Slice, reflect.Array:
if minLen := args.LengthNonIndexed(); v.Len() < minLen {
return fmt.Errorf("abi: insufficient number of elements in the list/array for unpack, want %d, got %d",
minLen, v.Len())
}
default:
return fmt.Errorf("abi: cannot unmarshal tuple into %v", t)
}
return nil
}