Proof Portal
jsonparser
ProbeLabs23 findings · 123 requirementsThe fastest JSON parser for Go — formally verified with ReqProof (real library, master).
When Delete calls internalGet and internalGet returns an error, Delete shall use that error to short-circuit to the safe fallback path (return original input unchanged) rather than discarding the error and proceeding with potentially invalid offsets.
Specification
The requirement exactly as authored — its complete prose text and, where present, the formal FRETish sentence it compiles to.
When Delete calls internalGet and internalGet returns an error, Delete shall use that error to short-circuit to the safe fallback path (return original input unchanged) rather than discarding the error and proceeding with potentially invalid offsets.
the parser shall always satisfy !delete_discards_internalGet_error | delete_propagates_internalGet_error
Rationale & tags
Why this requirement exists, and how it is categorised.
Rationale & tags
Why this requirement exists, and how it is categorised.
Error propagation from internalGet is the defense-in-depth mechanism against sentinel-value panics. If Delete discards the error (assigns to _), it loses the signal that offsets are invalid. This was a contributing factor in the PR #280 class.
Verification & provenance
How this requirement was checked: the review trail, edit history, and the machine-analysis status terms (each ⓘ explains what it means).
Review
- Status
- approved
- Reviewer
- Leonidbugaev
- Reviewed
- Apr 23, 2026, 00:00 UTC
History
- Created
- Apr 14, 2026, 18:00 UTC · Claude · AI agent
- Modified
- Jul 26, 2026, 14:54 UTC · Created via CLI
Change history
Every recorded revision of this requirement's source file — newest first, each with its commit message and the diff for that change.
Every recorded revision of this object's source — newest first, each with the commit message and its file-level diff.
Obligations
What this requirement must witness to be considered satisfied — the required evidence, and the tests that discharge each one.
1 obligation · 1 discharged
Browse the catalogueDelete discards the internalGet error and proceeds with stale offsets into data[prevTok]/data[end:] (parser.go:907), panicking with index-out-of-range on adversarial input — the OSS-Fuzz witness shape.
- nominalrequiredpresentCovered by 1 test
Formula evidence
The formal formula behind this requirement, the variables it is written over, and the tests that exercise it (each term is explained inline).
Formula evidence
The formal formula behind this requirement, the variables it is written over, and the tests that exercise it (each term is explained inline).
FRETish formula
the parser shall always satisfy !delete_discards_internalGet_error | delete_propagates_internalGet_error
Variables
| Name | Type | Direction | Description |
|---|---|---|---|
| delete_discards_internalGet_error | — | True when Delete discards an error returned by internalGet (assigns to underscore) instead of using it to short-circuit. | |
| delete_propagates_internalGet_error | — | True when Delete uses an error returned by internalGet to short-circuit to the safe fallback path. |
Witnesses· 3 scenarios total
- TestMCDC_SYS_REQ_049_Row1_TriggerFalseexercises 1 condition scenario
- TestMCDC_SYS_REQ_049_Row2_InvariantViolationexercises 1 condition scenario
- TestMCDC_SYS_REQ_049_Row3_PropagatedErrorexercises 1 condition scenario
MC/DC truth table· 3 rows
Each row assigns the formula's conditions (T/F) and shows the Result— the formula's value for that input row, not a test pass/fail. A row proves a condition when flipping only that condition flips the outcome. The test that covers each row is linked.
mcdc:ignoreNo-actionfalse-result row satisfied by designUncoveredneeds a covering test| # | delete_discards_internalGet_error | delete_propagates_internalGet_error | Result | Proves | Covering test |
|---|---|---|---|---|---|
| 1 | F | F | T | delete_discards_internalGet_error | |
| 2 | T | F | F | delete_discards_internalGet_error | |
| 3 | T | T | T | delete_propagates_internalGet_error |
Its place
This requirement shown inside its trace neighbourhood — the parents it satisfies, the code and tests attached to it, and its findings.
Loading graph…
Trace evidence
The concrete artifacts linked to this requirement — implementing code, verifying tests, documents, and the findings raised against it.
Impact
Blast radius — if you change this requirement, what else may need re-checking, and what it in turn depends on.
Impact
Blast radius — if you change this requirement, what else may need re-checking, and what it in turn depends on.
If you change this
Files to re-check (1)
- parser.goparser.go
Tests to re-run (3)
- deep_spec_test.godeep_spec_test.go
- mcdc_spec_witnesses_test.gomcdc_spec_witnesses_test.go
- obligation_evidence_test.goobligation_evidence_test.go
What this rests on
Parent requirements (1)
Changes
Semantic changes that move or scope this requirement — the intended work touching it. Each links to the full change.
Discussions
Discuss this with the proof team. Nothing changes in your audit automatically — you open a request and a staff member records any outcome inside the thread.