Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Jul 4, 2025. It is now read-only.
/cortex.cppPublic archive

bugfix: add more stringent out-of-bounds checks for GGUF parser#2159

Merged
gau-nernst merged 1 commit intodevfromthien/parser_oob
Mar 21, 2025

Conversation

@gau-nernst
Copy link
Contributor

Describe Your Changes

  • Check for out-of-boundswhenever we read a value from GGUF mmap file
  • Add a test that fails before this PR

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@gau-nernstgau-nernst changed the titlebug: add more stringent out-of-bounds checks for GGUF parserbugfix: add more stringent out-of-bounds checks for GGUF parserMar 21, 2025
@gau-nernstgau-nernst self-assigned thisMar 21, 2025
@github-project-automationgithub-project-automationbot moved this toInvestigating inMenloMar 21, 2025
@gau-nernstgau-nernst moved this fromInvestigating toEng Review inMenloMar 21, 2025
@gau-nernstgau-nernst merged commit3fef2db intodevMar 21, 2025
8 checks passed
@gau-nernstgau-nernst deleted the thien/parser_oob branchMarch 21, 2025 02:10
@david-menloaidavid-menloai moved this fromEng Review toCompleted inMenloMar 24, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

2 more reviewers

@vansangpfievvansangpfievvansangpfiev approved these changes

@qnixsynapseqnixsynapseqnixsynapse approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@gau-nernstgau-nernst

Labels

None yet

Projects

No open projects
Status: Completed

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gau-nernst@vansangpfiev@qnixsynapse

[8]ページ先頭

©2009-2025 Movatter.jp