Skip to content

fix(q): validate table wire markers - #9

Merged
protocolstardust merged 1 commit into
RayforceDB:masterfrom
belowzeroff:fix/q-table-marker-validation
Sep 25, 2026
Merged

protocolstardust merged 1 commit into
RayforceDB:masterfrom
belowzeroff:fix/q-table-marker-validation

Conversation

@belowzeroff

Copy link
Copy Markdown
Contributor

Summary

  • validate the table dictionary marker during q deserialization
  • propagate deserialization errors before trailing-byte checks
  • add a regression test for malformed table markers

Testing

  • codec self-test coverage added in test/driver.c

@protocolstardust
protocolstardust merged commit 810a23e into RayforceDB:master Sep 25, 2026
1 check passed
@protocolstardust

Copy link
Copy Markdown
Collaborator

Superseded by #10: its first commit is this exact change (810a23e), so it landed on master with #10 (e08ab96) and GitHub has marked this PR merged. Review notes are on #10.

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.

2 participants