Skip to content

Ensure ABI decoder rejects ZST offsets#359

Merged
q9f merged 2 commits intomainfrom
codex/review-zst-robustness-for-task-#311
Aug 6, 2025
Merged

Ensure ABI decoder rejects ZST offsets#359
q9f merged 2 commits intomainfrom
codex/review-zst-robustness-for-task-#311

Conversation

@q9f
Copy link
Owner

@q9f q9f commented Aug 6, 2025

  • validate dynamic array offsets to prevent self-referential ZST payloads
  • add regression spec covering malicious uint256[][] input

fix #311

@q9f q9f added the codex Codex is a durable, decentralised data storage protocol label Aug 6, 2025 — with ChatGPT Codex Connector
@codecov
Copy link

codecov bot commented Aug 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.58%. Comparing base (0fb08ee) to head (2fc19d9).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #359   +/-   ##
=======================================
  Coverage   99.58%   99.58%           
=======================================
  Files          87       87           
  Lines        6042     6055   +13     
=======================================
+ Hits         6017     6030   +13     
  Misses         25       25           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@q9f q9f merged commit 95bbcf4 into main Aug 6, 2025
8 checks passed
@q9f q9f deleted the codex/review-zst-robustness-for-task-#311 branch August 6, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex Codex is a durable, decentralised data storage protocol

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review ZST robustness

1 participant