dbgshim.dll!RegisterForRuntimeStartup can incorrectly fail with E_ACCESSDENIED on Windows

[gregg-miskelly]

RegisterForRuntimeStartup is requesting TOKEN_READ access when it should request TOKEN_QUERY access.

Here is the line of code with the bug: https://github.com/dotnet/coreclr/blob/32f0f9721afb584b4a14d69135bea7ddc129f755/src/utilcode/securityutil.cpp#L339

This prevents RegisterForRuntimeStartup from working correctly with Azure Service Fabric local simulator processes.

[lt72]

Are the local simulator processes an internal testing or a production scenario?

[gregg-miskelly]

That depends on your definition of 'production' -- this is how customers develop Azure Service Fabric applications on their dev box's. The problem might also exist with in-the-cloud debugging, but I am not sure if such a feature exists for Service Fabric.

[karelz]

BTW: I recommend to keep servicing issues opened, until they are merged into 2.0 servicing branch.

[Petermarcu]

This is approved for checkin when the branch is open for 2.0.2.

[Petermarcu]

This was merged for 2.0.1. Closing now.