Skip to content

fix(linux): fix memory problem 🍒 🏠#15828

Merged
ermshiperete merged 1 commit intostable-18.0from
fix/linux/cherry-pick/memmove
Apr 9, 2026
Merged

fix(linux): fix memory problem 🍒 🏠#15828
ermshiperete merged 1 commit intostable-18.0from
fix/linux/cherry-pick/memmove

Conversation

@ermshiperete
Copy link
Copy Markdown
Contributor

This fixes a problem identified by devin.ai: Because of the operator precedence the previous code caused memmove to read sizeof(commit_queue_item) - 1 bytes past the end of commit_queue. With this change memmove now reads the intended MAX_QUEUE_SIZE - 1 commit_queue_items.

Cherry-pick-of: #15823
Test-bot: skip

This fixes a problem identified by devin.ai: Because of the operator
precedence the previous code caused `memmove` to read
`sizeof(commit_queue_item) - 1` bytes past the end of `commit_queue`.
With this change `memmove` now reads the intended `MAX_QUEUE_SIZE - 1`
`commit_queue_items`.

Cherry-pick-of: #15823
Test-bot: skip
@ermshiperete ermshiperete requested a review from mcdurdin April 8, 2026 16:01
@github-project-automation github-project-automation Bot moved this to Todo in Keyman Apr 8, 2026
@github-actions github-actions Bot added linux/ linux/engine/ cherry-pick Change already merged into another (stable) branch fix stable labels Apr 8, 2026
@keymanapp-test-bot
Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

@keymanapp-test-bot keymanapp-test-bot Bot changed the title fix(linux): fix memory problem 🍒 fix(linux): fix memory problem 🍒 🏠 Apr 8, 2026
@keymanapp-test-bot keymanapp-test-bot Bot added this to the A19S26 milestone Apr 8, 2026
@ermshiperete ermshiperete merged commit f7c76d8 into stable-18.0 Apr 9, 2026
10 checks passed
@ermshiperete ermshiperete deleted the fix/linux/cherry-pick/memmove branch April 9, 2026 07:12
@github-project-automation github-project-automation Bot moved this from Todo to Done in Keyman Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick Change already merged into another (stable) branch fix linux/engine/ linux/ stable

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants