Skip to content

Fix missing AdminByRequest patches#4

Merged
Marc-André Moreau (mamoreau-devolutions) merged 1 commit intorelease/v2.6.1from
fix/adminbyrequest-patches
Apr 13, 2026
Merged

Fix missing AdminByRequest patches#4
Marc-André Moreau (mamoreau-devolutions) merged 1 commit intorelease/v2.6.1from
fix/adminbyrequest-patches

Conversation

@mamoreau-devolutions
Copy link
Copy Markdown
Contributor

Summary

Port the missing AdminByRequest-related source changes onto release/v2.6.1.

Problem

When UniGetUI Elevator was migrated into gsudo-distro, the AdminByRequest port was incomplete. The release branch kept the upstream StartElevatedDetached(...).GetSafeProcessHandle() path, which fails when AdminByRequest launches a transient shim process and that process exits before gsudo tries to access the handle.

Changes

  • stop relying on the elevated shell-launched Process object in the service start path
  • make StartElevatedDetached a fire-and-forget launch for the UAC-elevation path
  • remove the now-unused local service handle capture in RunCommand
  • evaluate UniElevate integrity checks with awareness of gsudoservice and gsudoelevate

Validation

  • built gsudo for net9.0 with /p:DisableGitVersionTask=true
  • built UniElevate with /p:BuildVariant=UniElevate /p:DisableGitVersionTask=true

Related: Devolutions/UniGetUI#4579, Devolutions/UniGetUI#3913

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant