Flight Log / OpenHFT/Chronicle-Bytes #687
Add bounds checks for array deserialisation — a OpenAI Codex (OpenAI) flight by Peter Lawrey
Peter Lawrey (@peter-lawrey) opened this OpenAI Codex (OpenAI) pull request in OpenHFT/Chronicle-Bytes on Jun 4, 2025; merged clean in 1.6 h, merged by @peter-lawrey: +86 −47, 4 files, 0 reviews.
Flown by
- Lines
- +86 −47
- Files
- 4
- Commits
- 2
- Reviews
- 0
- To merge
- 1.6 h
- Task
- feat
Line and file counts are summed per commit from the AIDev dataset; GitHub's net diff can be smaller.
The task was to add bounds checks for array deserialisation in Chronicle-Bytes. OpenAI Codex changed four files across two commits, adding a guard in BytesMarshaller and the system property bytes.max-array-len, documented in systemProperties.adoc. The test run failed: mvn was not found. It merged after 1.57 hours with no change requests and zero review comments.
From the pull request description
## Summary - guard against unreasonably large arrays in BytesMarshaller - expose new system property `bytes.max-array-len` - document the property in `systemProperties.adoc` - default the maximum array length to 16 MB instead of 1 MB ## Testing - `mvn -q verify` *(fails: `mvn` not found)* ------ h…
Excerpt (first 300 characters) — the full description is on GitHub.
View the pull request on GitHub →
This is your flight? Claim it — sign in with GitHub. Ownership is proven by the sign-in: the account that opened the PR is the only one that can claim it.
Flight timeline
- —committedPeter Lawrey
- —committedPeter Lawrey
- Jun 4, 2025labeledpeter-lawrey
- Jun 4, 2025mergedpeter-lawrey
- Jun 4, 2025closedpeter-lawrey
Read next
Cite this flight: AgentRiders Flight Log, OpenHFT/Chronicle-Bytes#687, record updated Nov 4, 2026, https://agentriders.com/logs/OpenHFT/Chronicle-Bytes/687/
Data from public GitHub activity; this page is not affiliated with GitHub or the repository.