]> Cypherpunks repositories - gostls13.git/commitdiff
runtime/race: also unmap memory if TestNonGoMemory fails
authorTobias Klauser <tklauser@distanz.ch>
Wed, 7 Sep 2022 11:35:24 +0000 (13:35 +0200)
committerGopher Robot <gobot@golang.org>
Fri, 9 Sep 2022 15:36:26 +0000 (15:36 +0000)
Change-Id: I7ea150e4896fc9b2e3a6dbdd9a1c2b651e74b844
Reviewed-on: https://go-review.googlesource.com/c/go/+/428778
Auto-Submit: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

src/runtime/race/race_unix_test.go

index 6cc0730589f33b37aa6a00699237d2df635f3a4b..3cf53b01871cf0fe7af64a08ca833cabe049c50d 100644 (file)
@@ -19,11 +19,11 @@ func TestNonGoMemory(t *testing.T) {
        if err != nil {
                t.Fatalf("failed to mmap memory: %v", err)
        }
+       defer syscall.Munmap(data)
        p := (*uint32)(unsafe.Pointer(&data[0]))
        atomic.AddUint32(p, 1)
        (*p)++
        if *p != 2 {
                t.Fatalf("data[0] = %v, expect 2", *p)
        }
-       syscall.Munmap(data)
 }