LBRYCommunity/lbcd
1
0
Fork 0
Code Issues 17 Pull requests 4 Projects Releases 28 Packages Wiki Activity

Fix SendManyCmd unmarshaling.

Browse source 
This change fixes an incorrect parameter length check, as well as
correcting the type assertions for the address to amount pairs.
This commit is contained in:
Josh Rickmar 2013-11-12 11:24:33 -05:00
parent 56a62309ae
commit d3dea375c6
1 changed files with 9 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
jsoncmd.go
View file

@ -5101,7 +5101,7 @@ func (cmd *SendManyCmd) UnmarshalJSON(b []byte) error {
return err
}
if len(r.Params) > 6 || len(r.Params) < 3 {
if len(r.Params) > 4 || len(r.Params) < 2 {
return ErrWrongNumberOfParams
}
@ -5110,15 +5110,19 @@ func (cmd *SendManyCmd) UnmarshalJSON(b []byte) error {
return errors.New("first parameter fromaccount must be a string")
}
famounts, ok := r.Params[1].(map[string]float64)
iamounts, ok := r.Params[1].(map[string]interface{})
if !ok {
return errors.New("second parameter toaccount must be a string to number map")
return errors.New("second parameter toaccount must be a JSON object")
}
amounts := make(map[string]int64)
for k, v := range famounts {
for k, v := range iamounts {
famount, ok := v.(float64)
if !ok {
return errors.New("second parameter toaccount must be a string to number map")
}
var err error
amounts[k], err = JSONToAmount(v)
amounts[k], err = JSONToAmount(famount)
if err != nil {
return err
}