Repository navigation
Leading whitespaces lead to incorrect results (jsonparser.ArrayEach) #150
Description
data := []byte(`[1, 2, 3]`)
jsonparser.ArrayEach(data, func(value []byte, dataType jsonparser.ValueType, offset int, err error) {
fmt.Println(string(value))
})
returns
1
2
3
but
data := []byte(` [1, 2, 3]`)
jsonparser.ArrayEach(data, func(value []byte, dataType jsonparser.ValueType, offset int, err error) {
fmt.Println(string(value))
})
returns
[1, 2, 3]
Activity
sauerbraten commentedon Jan 22, 2021
Similarly, Delete seems to "silently" fail when there is leading whitespace:
This works: https://play.golang.org/p/Jub6y91ObT0
This doesn't: https://play.golang.org/p/FLM_k3GU6nc
buger commentedon Jan 22, 2021
Should be a pretty simple fix, e.g. just skip spaces at the start.
PRs are welcomed :)
buger commentedon Mar 19, 2021
Fixed now, thanks @daria-kay!
buger commentedon Mar 21, 2021
Have to re-open because of bug found by OSS-Fuzz. See attached PR for error details.
kadukm commentedon May 11, 2021
OSS-Fuzz bug is solved by #227. Can issue be closed?
buger commentedon Jul 28, 2026
Fixed — ArrayEach now handles leading whitespace correctly. Additionally, v1.3.0 hardened ArrayEach to return a clean error (no spurious callback) when the root value is not an array.