diff options
| author | Ian Lance Taylor <iant@golang.org> | 2025-10-28 13:35:53 -0700 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2025-11-04 16:32:25 -0800 |
| commit | f5f14262d0c0aae7d104ce22d05862276d16a002 (patch) | |
| tree | 7c98e7586b337fcf99f5ccf5e09239ec22a52963 /src/cmd/link | |
| parent | 61de3a9dae6d9e7b0b2d31ec2650ee88237d8bfb (diff) | |
| download | go-f5f14262d0c0aae7d104ce22d05862276d16a002.tar.xz | |
cmd/link: remove misleading comment
The comment suggests that the text section is briefly writable.
That is not the case. As the earlier part of the comment explains,
part of the text section is mapped twice, once r-x and once rw-.
It is never the case that there is writable executable memory.
Change-Id: I56841e19a8a08f2515f29752536a5c8f180ac8c9
Reviewed-on: https://go-review.googlesource.com/c/go/+/715622
Auto-Submit: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Cherry Mui <cherryyz@google.com>
Reviewed-by: Than McIntosh <thanm@golang.org>
Reviewed-by: Michael Pratt <mpratt@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Diffstat (limited to 'src/cmd/link')
| -rw-r--r-- | src/cmd/link/internal/ld/data.go | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/cmd/link/internal/ld/data.go b/src/cmd/link/internal/ld/data.go index b70d050c99..a8e56bdd5a 100644 --- a/src/cmd/link/internal/ld/data.go +++ b/src/cmd/link/internal/ld/data.go @@ -2852,9 +2852,6 @@ func (ctxt *Link) address() []*sym.Segment { // will be such that the last page of the text segment will be // mapped twice, once r-x and once starting out rw- and, after // relocation processing, changed to r--. - // - // Ideally the last page of the text segment would not be - // writable even for this short period. va = uint64(Rnd(int64(va), *FlagRound)) order = append(order, &Segrodata) |