Skip to content

fix: allocate additional_drivers with correct element type#1909

Merged
charles-lunarg merged 1 commit into
KhronosGroup:mainfrom
aizu-m:additional-drivers-elem-size
Jun 1, 2026
Merged

fix: allocate additional_drivers with correct element type#1909
charles-lunarg merged 1 commit into
KhronosGroup:mainfrom
aizu-m:additional-drivers-elem-size

Conversation

@aizu-m
Copy link
Copy Markdown
Contributor

@aizu-m aizu-m commented Jun 1, 2026

parse_additional_drivers at loader/settings.c:270 sizes the additional_drivers array with sizeof(loader_settings_layer_configuration), but its elements are loader_settings_driver_configuration. Use the matching type so the allocation size matches what's indexed and freed.

@ci-tester-lunarg
Copy link
Copy Markdown

Author aizu-m not on autobuild list. Waiting for curator authorization before starting CI build.

1 similar comment
@ci-tester-lunarg
Copy link
Copy Markdown

Author aizu-m not on autobuild list. Waiting for curator authorization before starting CI build.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jun 1, 2026

CLA assistant check
All committers have signed the CLA.

@ci-tester-lunarg
Copy link
Copy Markdown

CI Vulkan-Loader build queued with queue ID 752264.

@ci-tester-lunarg
Copy link
Copy Markdown

CI Vulkan-Loader build # 3495 running.

@ci-tester-lunarg
Copy link
Copy Markdown

CI Vulkan-Loader build # 3495 passed.

@charles-lunarg charles-lunarg merged commit 70e3d87 into KhronosGroup:main Jun 1, 2026
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants