diff options
author | Alex Brainman <alex.brainman@gmail.com> | 2016-04-05 11:31:50 +1000 |
---|---|---|
committer | Andrew Gerrand <adg@golang.org> | 2016-04-08 05:11:52 +0000 |
commit | 90bf0f1e5dd1023034142efb72021a9bacb418ce (patch) | |
tree | c6979573336cbaf4ecf745cb81c11daf39669cd5 | |
parent | 77ef9c7e79a8e8d948c11330584ea205b35bb0e1 (diff) | |
download | go-90bf0f1e5dd1023034142efb72021a9bacb418ce.tar.gz go-90bf0f1e5dd1023034142efb72021a9bacb418ce.zip |
runtime: leave directory before removing it in TestDLLPreloadMitigation
Fixes #15120
Change-Id: I1d9a192ac163826bad8b46e8c0b0b9e218e69570
Reviewed-on: https://go-review.googlesource.com/21520
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-on: https://go-review.googlesource.com/21681
Run-TryBot: Andrew Gerrand <adg@golang.org>
Reviewed-by: Alex Brainman <alex.brainman@gmail.com>
-rw-r--r-- | src/runtime/syscall_windows_test.go | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/src/runtime/syscall_windows_test.go b/src/runtime/syscall_windows_test.go index ee449f93f7..6a5e536b6a 100644 --- a/src/runtime/syscall_windows_test.go +++ b/src/runtime/syscall_windows_test.go @@ -649,6 +649,17 @@ func TestDLLPreloadMitigation(t *testing.T) { t.Skip("skipping test: gcc is missing") } + tmpdir, err := ioutil.TempDir("", "TestDLLPreloadMitigation") + if err != nil { + t.Fatal("TempDir failed: ", err) + } + defer func() { + err := os.RemoveAll(tmpdir) + if err != nil { + t.Error(err) + } + }() + dir0, err := os.Getwd() if err != nil { t.Fatal(err) @@ -663,12 +674,6 @@ uintptr_t cfunc() { SetLastError(123); } ` - tmpdir, err := ioutil.TempDir("", "TestDLLPreloadMitigation") - if err != nil { - t.Fatal("TempDir failed: ", err) - } - defer os.RemoveAll(tmpdir) - srcname := "nojack.c" err = ioutil.WriteFile(filepath.Join(tmpdir, srcname), []byte(src), 0) if err != nil { |