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

vulkaninfo: On X11 use default visual #1042

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

rp42
Copy link
Contributor

@rp42 rp42 commented Oct 23, 2024

If the visual passed to XCreateWindow is not the default one, it can cause a BadMatch error. Usually the first one returned does work, but in certain cases we need to search through all the visuals to find one that matches the default visual. This was seen on an xrdp session on Rocky 9.

Fixes #1042

If the visual passed to XCreateWindow is not the default one, it can
cause a BadMatch error. Usually the first one returned does work, but in
certain cases we need to search through all the visuals to find one that
matches the default visual. This was seen on an xrdp session on Rocky 9.
@ci-tester-lunarg
Copy link

Author rp42 not on autobuild list. Waiting for curator authorization before starting CI build.

1 similar comment
@ci-tester-lunarg
Copy link

Author rp42 not on autobuild list. Waiting for curator authorization before starting CI build.

@CLAassistant
Copy link

CLAassistant commented Oct 23, 2024

CLA assistant check
All committers have signed the CLA.

@rp42 rp42 marked this pull request as ready for review October 23, 2024 14:53
@ci-tester-lunarg
Copy link

CI Vulkan-Tools build queued with queue ID 285339.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build # 1552 running.

@ci-tester-lunarg
Copy link

CI Vulkan-Tools build # 1552 passed.

@charles-lunarg charles-lunarg merged commit a886096 into KhronosGroup:main Oct 23, 2024
18 checks passed
@charles-lunarg
Copy link
Contributor

Thank you again for the fix!

@charles-lunarg charles-lunarg linked an issue Oct 23, 2024 that may be closed by this pull request
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.

BadMatch on X_CreateWindow running on xrdp on Rocky 9
4 participants