tree 6f2f34c961667ca763ee8436d4e52a572a80bb6b
parent d9b008d0a637a6daa9d387171bc2d206f4de1691
author Jason A. Donenfeld <Jason@zx2c4.com> 1605648590 +0100
committer Jason A. Donenfeld <Jason@zx2c4.com> 1605651995 +0000

windows/svc/mgr: fetch service sid type in Config()

Currently, doing s.UpdateConfig(s.Config()) will destroy the service,
because s.Config() fails to populate the SidType member, but
UpdateConfig will set the SidType, so that expression effectively zeros
out the SidType. Fix this by having Config() fetch the SidType in the
same way that it fetches the other additional fields there, such as
DelayedStartUp.

Change-Id: Idb917ef1e942020499b411b7777b995c29f0e7d2
Reviewed-on: https://go-review.googlesource.com/c/sys/+/270897
Trust: Jason A. Donenfeld <Jason@zx2c4.com>
Run-TryBot: Jason A. Donenfeld <Jason@zx2c4.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>