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: fallback to strings for non-executable libc.so.6 #202581

Conversation

khreenberg
Copy link
Contributor

This PR adds a fallback to strings on hosts where libc.so.6 is not executable. I don't know if using strings is a robust solution, but it resolves the issue for me on WSL distro Ubuntu-Preview.

Fixes #202580

@xangma
Copy link

xangma commented Jan 16, 2024

Just to confirm, this is also happening on Ubuntu 22 and 23. Strings does seem to get the right info though :-)

@alexr00 alexr00 assigned deepak1556 and unassigned alexr00 Jan 16, 2024
@khreenberg khreenberg force-pushed the fallback-to-strings-for-non-executable-libc branch from cee0a47 to 77cface Compare January 16, 2024 13:53
@deepak1556 deepak1556 added this to the December / January 2024 milestone Jan 16, 2024
@khreenberg khreenberg force-pushed the fallback-to-strings-for-non-executable-libc branch from 77cface to 6a606e3 Compare January 16, 2024 16:56
Copy link
Contributor

@deepak1556 deepak1556 left a comment

Choose a reason for hiding this comment

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

Thanks for catching this issue, left a minor change.

resources/server/bin/helpers/check-requirements-linux.sh Outdated Show resolved Hide resolved
@deepak1556 deepak1556 force-pushed the fallback-to-strings-for-non-executable-libc branch from 35466c8 to 63f3dcc Compare January 17, 2024 02:30
@deepak1556 deepak1556 enabled auto-merge (squash) January 17, 2024 02:31
@deepak1556 deepak1556 merged commit aed5f4d into microsoft:main Jan 17, 2024
6 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.

WSL Server unable to start if 'libc.so.6' is not executable
6 participants