Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix swapchain image wrappers lifetime #1424

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

ziga-lunarg
Copy link
Contributor

If an application creates multiple swapchains, the images retrieved with vkGetSwapchainImages() can be the same for different swapchains. The parents of swapchain images were not tracked and when one swapchain was destroyed the swapchain image wrappers were as well, even though another swapchain might still be using them

If an application creates multiple swapchains, the images retrieved with
vkGetSwapchainImages() can be the same for different swapchains. The
parents of swapchain images were not tracked and when one swapchain was
destroyed the swapchain image wrappers were as well, even though another
swapchain might still be using them
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 134561.

@CLAassistant
Copy link

CLAassistant commented Feb 16, 2024

CLA assistant check
All committers have signed the CLA.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3735 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3735 passed.

@locke-lunarg
Copy link
Contributor

locke-lunarg commented Feb 16, 2024

@davidlunarg has a draft PR #1304 to fix a swapchain issue for VK_OBS_HOOK. It's kind of similar case.

Copy link
Contributor

@locke-lunarg locke-lunarg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ziga-lunarg ziga-lunarg merged commit b78f35e into LunarG:dev Feb 21, 2024
8 checks passed
@ziga-lunarg ziga-lunarg deleted the fix-swapchain-images branch February 21, 2024 16:07
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