diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2015-01-30 14:36:12 +0300 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2015-01-30 12:01:31 +0000 |
| commit | 3c3848ad92bac9edce2ec1e510c01f9bf2317ea3 (patch) | |
| tree | 3f0294723e4dbc5074a9803e157a8eaf7bdf254e /src/runtime/pprof | |
| parent | 256116ad2564b10c18750633dce047fd92d57bc7 (diff) | |
| download | go-3c3848ad92bac9edce2ec1e510c01f9bf2317ea3.tar.xz | |
runtime: fix system memory allocator on plan9
The following line in sysFree:
n += (n + memRound) &^ memRound
doubles value of n (n += n).
Which is wrong and can lead to memory corruption.
Fixes #9712
Change-Id: I3c141b71da11e38837c09408cf4f1d22e8f7f36e
Reviewed-on: https://go-review.googlesource.com/3602
Reviewed-by: David du Colombier <0intro@gmail.com>
Diffstat (limited to 'src/runtime/pprof')
| -rw-r--r-- | src/runtime/pprof/trace_test.go | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/runtime/pprof/trace_test.go b/src/runtime/pprof/trace_test.go index c42032455c..1b99830bfa 100644 --- a/src/runtime/pprof/trace_test.go +++ b/src/runtime/pprof/trace_test.go @@ -20,8 +20,6 @@ func skipTraceTestsIfNeeded(t *testing.T) { switch runtime.GOOS { case "solaris": t.Skip("skipping: solaris timer can go backwards (http://golang.org/issue/8976)") - case "plan9": - t.Skip("skipping: plan9 tests fail with out of memory (http://golang.org/issue/9712") } switch runtime.GOARCH { @@ -234,7 +232,7 @@ eventLoop: for _, f := range ev.stk { if strings.HasSuffix(f.file, "trace_test.go") && strings.HasSuffix(f.fn, "pprof_test.TestTraceSymbolize") && - f.line == 218 { + f.line == 216 { found = true break eventLoop } |