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 directory selector crashing when attempting to display a bitlocker locked drive #5478

Merged
merged 2 commits into from
Oct 24, 2022

Conversation

@smoogipoo
Copy link
Contributor

Should probably display this error somewhere/give the error somewhere for a consumer to handle and display it, but probably fine for now?

@smoogipoo smoogipoo requested a review from bdach October 24, 2022 04:31
@bdach
Copy link
Collaborator

bdach commented Oct 24, 2022

Can't test the exact scenario on my windows install because it can't into bitlocker but I can't imagine this not fixing it.

I suggested logging the error in the issue, but I think no logging is fine, given that UnauthorizedAccessException has been silenced already and nobody has complained since.

@bdach bdach enabled auto-merge October 24, 2022 17:56
@bdach bdach merged commit 603e15f into ppy:master Oct 24, 2022
@peppy peppy deleted the fix-bitlocker-crash branch November 7, 2022 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants