Panic in jsonb.rs: "attempt to add with overflow"

[LeMikaelF]

Seed: 337756940

cargo +nightly fuzz run scalar_func -- -max_total_time=300

...

thread '<unnamed>' (5692) panicked at /home/runner/_work/turso/turso/core/json/jsonb.rs:2975:16:
attempt to add with overflow

Failing job: https://github.com/tursodatabase/turso/actions/runs/21767747415/job/62807840849?pr=4680

[oopscompiled]

Hi, I've analysed the issue and it seems like there were some unchecked addition operations in (skip_element, validate_element, and object serialization) that causes the overflow, so something like .checked_add(skip_header) will safely return a parse error instead of panicking.

I can send a PR with these changes if you'd like.

[LeMikaelF]

@oopscompiled Please do!

Have a look at docs/agent-guides/testing.md for our testing strategy. We aim at full SQLite compatibility, so the fix to this should include a SQL test in json/memory.sqltest. You can then run the full test suite to ensure Turso has the same behaviour as SQLite with make -C testing/runner run-cli (for turso) and make -C testing/runner test-sqlite (for sqlite).

[LeMikaelF]

@oopscompiled are you still working on this?

[oopscompiled]

@LeMikaelF Hi! the PR is here: #5078

[eduardorittner]

this issue has been fixed as of #5061, right?