Skip to content

Add bounds checks to JSON Lines array iterator#10

Open
SRO-SA wants to merge 1 commit into
masterfrom
fix/issue-5-jsonlines-gotoarrayindex-bounds
Open

Add bounds checks to JSON Lines array iterator#10
SRO-SA wants to merge 1 commit into
masterfrom
fix/issue-5-jsonlines-gotoarrayindex-bounds

Conversation

@SRO-SA

@SRO-SA SRO-SA commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Fixes #5

Summary:

  • Adds bounds checks before JSON Lines iterator traversal reads structural-array entries.
  • Validates node, nextNode, pair_pos, and structural-derived file positions before access.
  • Avoids mutating iterator state until the requested array index is confirmed valid.
  • Rejects negative indexes and empty-array accesses.

Testing:

  • I could not run the ASAN reproducer locally because my laptop does not have CUDA access.
  • I reviewed the affected host-side iterator traversal manually.
  • Please rerun the ASAN reproducer from the issue to confirm the heap-buffer-overflow is resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Heap buffer overflow in cuJSONLinesIterator::gotoArrayIndex

1 participant