Skip to content

Fix out-of-bounds access to override BOBs ny-offset - #1984

Merged
Flow86 merged 1 commit into
Return-To-The-Roots:masterfrom
Flamefire:fix-bob-crash
Sep 28, 2026
Merged

Flow86 merged 1 commit into
Return-To-The-Roots:masterfrom
Flamefire:fix-bob-crash

Conversation

@Flamefire

Copy link
Copy Markdown
Member

In glArchivItem_Bob::mergeLinks the ny-offset array for additional Bobs was not resized leading to an OOB access when accessing the sprites after #1968
Fix by using a single array instead.

Fixes #1982
Fixes #1983

@Flamefire
Flamefire requested a review from Flow86 September 23, 2026 07:15
@Flamefire
Flamefire enabled auto-merge (rebase) September 24, 2026 17:18
In `glArchivItem_Bob::mergeLinks` the ny-offset array for additional Bobs was not resized leading to an OOB access when accessing the sprites
after Return-To-The-Roots#1968
Fix by using a single array instead.

Fixes Return-To-The-Roots#1982
Fixes Return-To-The-Roots#1983
@Flow86
Flow86 disabled auto-merge September 28, 2026 19:47
@Flow86
Flow86 merged commit a131e83 into Return-To-The-Roots:master Sep 28, 2026
20 of 21 checks passed
@Flamefire
Flamefire deleted the fix-bob-crash branch September 29, 2026 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants