1
0
mirror of https://github.com/golang/go synced 2024-11-21 21:24:45 -07:00
go/src
Michael Anthony Knyszek be062b7f61 [release-branch.go1.23] runtime: explicitly keep handle alive during getOrAddWeakHandle
getOrAddWeakHandle is very careful about keeping its input alive across
the operation, but not very careful about keeping the heap-allocated
handle it creates alive. In fact, there's a window in this function
where it is *only* visible via the special. Specifically, the window of
time between when the handle is stored in the special and when the
special actually becomes visible to the GC.

(If we fail to add the special because it already exists, that case is
fine. We don't even use the same handle value, but the one we obtain
from the attached GC-visible special, *and* we return that value, so it
remains live.)

For #70455.
Fixes #70469.

Change-Id: Iadaff0cfb93bcaf61ba2b05be7fa0519c481de82
Reviewed-on: https://go-review.googlesource.com/c/go/+/630316
Auto-Submit: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-11-20 19:59:34 +00:00
..
archive archive/zip: document handling of duplicate names in Writer.Create 2024-07-16 18:02:59 +00:00
arena
bufio
builtin
bytes internal/bytealg: extend memchr result correctly on wasm 2024-07-17 07:00:20 +00:00
cmd [release-branch.go1.23] cmd/compile: use a non-fragile test for "does f contain closure c?" 2024-11-19 17:52:28 +00:00
cmp
compress
container
context context: handle nil values for valueCtx.String() 2024-07-10 02:44:20 +00:00
crypto [release-branch.go1.23] crypto/tls: fix testHandshake close flakes 2024-08-02 19:55:18 +00:00
database/sql [release-branch.go1.23] database/sql: fix panic with concurrent Conn and Close 2024-08-28 18:37:00 +00:00
debug all: document legacy //go:linkname for final round of modules 2024-05-29 17:58:53 +00:00
embed embed: simplify the code 2024-05-20 19:42:51 +00:00
encoding [release-branch.go1.23] encoding/gob: cover missed cases when checking ignore depth 2024-09-05 14:55:14 +00:00
errors errors: change interface{} to any in comment 2024-05-24 17:13:04 +00:00
expvar
flag flag: add FlagSet example 2024-07-15 21:12:53 +00:00
fmt all: change from sort functions to slices functions where feasible 2024-05-23 01:00:11 +00:00
go [release-branch.go1.23] go/build/constraint: add parsing limits 2024-09-05 14:55:17 +00:00
hash all: make function comments match function names 2024-06-03 14:56:25 +00:00
html html: update jstmpllitinterp doc 2024-05-22 18:04:29 +00:00
image all: make function comments match function names 2024-06-03 14:56:25 +00:00
index/suffixarray all: change from sort functions to slices functions where feasible 2024-05-23 01:00:11 +00:00
internal [release-branch.go1.23] internal/poll: handle the special case of sendfile(2) sending the full chunk 2024-10-30 17:04:03 +00:00
io io: add test for Pipe constructor allocations 2024-07-07 19:43:22 +00:00
iter iter: minor doc comment updates 2024-06-21 19:12:59 +00:00
log all: make struct comments match struct names 2024-07-11 17:23:45 +00:00
maps maps: document handling of non-reflexive keys 2024-07-17 23:08:52 +00:00
math math/big: fix comment typo in natdiv.go 2024-07-17 15:08:04 +00:00
mime all: document legacy //go:linkname for final round of modules 2024-05-29 17:58:53 +00:00
net net: document ParseIP always returns IPv6 addresses 2024-07-16 18:03:01 +00:00
os [release-branch.go1.23] internal/poll: keep copying after successful Sendfile return on BSD 2024-10-30 17:03:55 +00:00
path all: change from sort functions to slices functions where feasible 2024-05-23 01:00:11 +00:00
plugin
reflect all: document legacy //go:linkname for final round of modules 2024-05-29 17:58:53 +00:00
regexp regexp: more cross-references in docstrings 2024-07-16 20:07:41 +00:00
runtime [release-branch.go1.23] runtime: explicitly keep handle alive during getOrAddWeakHandle 2024-11-20 19:59:34 +00:00
slices slices: explicitly document nil and empty slice equality 2024-07-22 17:38:46 +00:00
sort sort: add example for Find 2024-07-16 17:55:15 +00:00
strconv
strings cmd/compile: don't inline runtime functions in -d=checkptr build 2024-07-22 15:45:09 +00:00
structs cmd/compile: add structs.HostLayout 2024-05-20 21:19:39 +00:00
sync all: make struct comments match struct names 2024-07-11 17:23:45 +00:00
syscall [release-branch.go1.23] syscall: mark SyscallN as noescape 2024-11-20 18:07:04 +00:00
testdata
testing testing: remove call to os.Exit in documentation for TestMain 2024-07-08 22:10:16 +00:00
text text/template/parse: fix handling of assignment/declaration in PipeNode.String 2024-06-21 18:12:29 +00:00
time [release-branch.go1.23]time: accept "+01" in TestLoadFixed on OpenBSD 2024-11-19 18:04:02 +00:00
unicode
unique [release-branch.go1.23] unique: don't retain uncloned input as key 2024-09-11 18:11:20 +00:00
unsafe unsafe: say "functions like syscall.Syscall", not only Syscall 2024-07-11 23:38:31 +00:00
vendor cmd: vendor golang.org/x/telemetry@268b4a8ec2d7 2024-07-12 22:42:17 +00:00
all.bash
all.bat
all.rc
bootstrap.bash
buildall.bash
clean.bash
clean.bat
clean.rc
cmp.bash
go.mod cmd: vendor golang.org/x/telemetry@268b4a8ec2d7 2024-07-12 22:42:17 +00:00
go.sum cmd: vendor golang.org/x/telemetry@268b4a8ec2d7 2024-07-12 22:42:17 +00:00
make.bash make.bash: preserve GOROOT_BOOTSTRAP 2024-05-29 13:48:46 +00:00
make.bat
Make.dist
make.rc make.bash: preserve GOROOT_BOOTSTRAP 2024-05-29 13:48:46 +00:00
race.bash
race.bat
README.vendor
run.bash run.bash: rm bumping open files soft limit 2024-05-15 15:02:23 +00:00
run.bat
run.rc

Vendoring in std and cmd
========================

The Go command maintains copies of external packages needed by the
standard library in the src/vendor and src/cmd/vendor directories.

There are two modules, std and cmd, defined in src/go.mod and
src/cmd/go.mod. When a package outside std or cmd is imported
by a package inside std or cmd, the import path is interpreted
as if it had a "vendor/" prefix. For example, within "crypto/tls",
an import of "golang.org/x/crypto/cryptobyte" resolves to
"vendor/golang.org/x/crypto/cryptobyte". When a package with the
same path is imported from a package outside std or cmd, it will
be resolved normally. Consequently, a binary may be built with two
copies of a package at different versions if the package is
imported normally and vendored by the standard library.

Vendored packages are internally renamed with a "vendor/" prefix
to preserve the invariant that all packages have distinct paths.
This is necessary to avoid compiler and linker conflicts. Adding
a "vendor/" prefix also maintains the invariant that standard
library packages begin with a dotless path element.

The module requirements of std and cmd do not influence version
selection in other modules. They are only considered when running
module commands like 'go get' and 'go mod vendor' from a directory
in GOROOT/src.

Maintaining vendor directories
==============================

Before updating vendor directories, ensure that module mode is enabled.
Make sure that GO111MODULE is not set in the environment, or that it is
set to 'on' or 'auto', and if you use a go.work file, set GOWORK=off.

Requirements may be added, updated, and removed with 'go get'.
The vendor directory may be updated with 'go mod vendor'.
A typical sequence might be:

    cd src  # or src/cmd
    go get golang.org/x/net@master
    go mod tidy
    go mod vendor

Use caution when passing '-u' to 'go get'. The '-u' flag updates
modules providing all transitively imported packages, not only
the module providing the target package.

Note that 'go mod vendor' only copies packages that are transitively
imported by packages in the current module. If a new package is needed,
it should be imported before running 'go mod vendor'.