]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: uses SID of group Guests to test ACL
authorConstantin Konstantinidis <constantinkonstantinidis@gmail.com>
Sat, 28 Jul 2018 08:16:40 +0000 (10:16 +0200)
committerAlex Brainman <alex.brainman@gmail.com>
Sat, 28 Jul 2018 09:06:26 +0000 (09:06 +0000)
The test TestACL failed when ran on a Windows set
up in another language as the "Guest" account
name is translated. The SID of the group of Guests
always exist and is used instead.

Fixes #26658

Change-Id: Ia885d08a9e50563787e389c2d2dc2547881a2943
Reviewed-on: https://go-review.googlesource.com/126598
Reviewed-by: Alex Brainman <alex.brainman@gmail.com>
Run-TryBot: Alex Brainman <alex.brainman@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/cmd/go/go_windows_test.go

index f278741c8bc12f262cc6eff557ec9cc60a2569cc..99af3d43dccb431a2828a4f086454e3656b910f6 100644 (file)
@@ -97,7 +97,7 @@ func TestACL(t *testing.T) {
        // will make all files created in TestACL/tmp have different
        // security attributes to the files created in TestACL.
        runIcacls(t, newtmpdir,
-               "/grant", "guest:(oi)(ci)f", // add Guest user to have full access
+               "/grant", "*S-1-5-32-546:(oi)(ci)f", // add Guests group to have full access
        )
 
        src := filepath.Join(tmpdir, "main.go")