status-im/nim-json-rpc
2
0
Fork 0
mirror of https://github.com/status-im/nim-json-rpc.git synced 2025-02-24 10:18:15 +00:00
Code Issues Projects Releases Wiki Activity

Fix off by one error in runtime check for array length

Browse Source
This commit is contained in:
coffeepots 2018-05-01 20:59:10 +01:00 committed by zah
parent f4598f563e
commit 562eb71ee5
1 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
eth-rpc/server/servertypes.nim
View File

@ -159,13 +159,17 @@ macro processFields(jsonIdent, fieldName, fieldType: typed): untyped =
else:
# array
let
expectedParams = typeInfo[1][1][2]
expectedParamsStr = expectedParams.toStrLit
expectedLenStr = "Expected parameter `" & fieldName.repr & "` to have a length of " & $expectedParamsStr & " but got "
startLen = typeInfo[1][1][1]
endLen = typeInfo[1][1][2]
expectedLen = genSym(nskConst)
expectedParams = quote do:
const `expectedLen` = `endLen` - `startLen` + 1
expectedLenStr = "Expected parameter `" & fieldName.repr & "` to have a length of "
# TODO: Note, currently only raising if greater than value, not different size
result.add(quote do:
if `jsonIdent`.len > `expectedParams`:
raise newException(ValueError, `expectedLenStr` & $`jsonIdent`.len)
`expectedParams`
if `jsonIdent`.len > `expectedLen`:
raise newException(ValueError, `expectedLenStr` & $`expectedLen` & " but got " & $`jsonIdent`.len)
)
result.add(quote do:
@ -257,7 +261,7 @@ when isMainModule:
res.add %int(item)
result = res
type Test = object
a: int #array[0..10, int]
a: seq[int] #array[0..10, int]
type MyObject* = object
a: int
@ -282,7 +286,7 @@ when isMainModule:
check r2 == ckR2
test "Object parameters":
let
obj = %*{"a": %1, "b": %*{"a": %5}, "c": %1.23}
obj = %*{"a": %1, "b": %*{"a": %[5]}, "c": %1.23}
r = waitfor rpcObjParam(%[%"abc", obj])
check r == obj
test "Runtime errors":