diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2019-11-27 11:17:35 +0100 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2019-12-11 12:06:25 +0100 |
commit | fb6200a30dbc54f249f26d67eb5c7584fe51c468 (patch) | |
tree | aa2d970adc38639f16b6d309b48afdb201dc8a64 /conf | |
parent | tunnel: add wintun ordered unit test (diff) | |
download | wireguard-windows-fb6200a30dbc54f249f26d67eb5c7584fe51c468.tar.xz wireguard-windows-fb6200a30dbc54f249f26d67eb5c7584fe51c468.zip |
conf: migrate when windows.old is S-1-5-32-544
Reported-by: Mantas Mikulėnas <grawity@gmail.com>
Diffstat (limited to 'conf')
-rw-r--r-- | conf/migration_windows.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/conf/migration_windows.go b/conf/migration_windows.go index fb12eaf0..72b298b6 100644 --- a/conf/migration_windows.go +++ b/conf/migration_windows.go @@ -36,8 +36,8 @@ func maybeMigrate(c string) { log.Printf("Not migrating configuration from ‘%s’ due to GetSecurityDescriptorOwner error: %v", oldRoot, err) return } - if defaulted || !owner.IsWellKnown(windows.WinLocalSystemSid) { - log.Printf("Not migrating configuration from ‘%s’, as it is not explicitly owned by SYSTEM, but rather ‘%v’", oldRoot, owner) + if defaulted || (!owner.IsWellKnown(windows.WinLocalSystemSid) && !owner.IsWellKnown(windows.WinBuiltinAdministratorsSid)) { + log.Printf("Not migrating configuration from ‘%s’, as it is not explicitly owned by SYSTEM or Built-in Administrators, but rather ‘%v’", oldRoot, owner) return } err = windows.MoveFileEx(windows.StringToUTF16Ptr(oldC), windows.StringToUTF16Ptr(c), windows.MOVEFILE_COPY_ALLOWED) |