]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.21] cmd/go: reject toolchain directives containing path separators
authorBryan C. Mills <bcmills@google.com>
Thu, 17 Aug 2023 21:12:44 +0000 (17:12 -0400)
committerCherry Mui <cherryyz@google.com>
Wed, 6 Sep 2023 14:19:53 +0000 (14:19 +0000)
commitd25a935574efd573668d8ce9ea4cfc530bb63ecb
treec4ec76fc4ab6f543dcfa39f46f4c347a0da7c994
parente3ba569c78d2c496537f4546282a5e04f33c886e
[release-branch.go1.21] cmd/go: reject toolchain directives containing path separators

If GOTOOLCHAIN="path" or "auto", the go command uses exec.LookPath to
search for it in order to allow toolchains to refer to local-only
toolchain variants (such as toolchains built from enterprise- or
distro-patched source). However, those toolchains should only be
resolved from $PATH, not relative to the working directory of the
command.

Thanks to Juho Nurminen of Mattermost for reporting this issue.

Fixes #62198.
Fixes #62394.
Fixes CVE-2023-39320.

Change-Id: I247c7acea95d737362dd0475e9fc8515430d0fcc
Reviewed-on: https://team-review.git.corp.google.com/c/golang/go-private/+/1996318
Run-TryBot: Roland Shoemaker <bracewell@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
Reviewed-by: Roland Shoemaker <bracewell@google.com>
(cherry picked from commit e41c0a55d45e9a9acbc5d7c1143ea4fff8fb9283)
Reviewed-on: https://team-review.git.corp.google.com/c/golang/go-private/+/2014013
Reviewed-by: Bryan Mills <bcmills@google.com>
Reviewed-by: Tatiana Bradley <tatianabradley@google.com>
Reviewed-on: https://go-review.googlesource.com/c/go/+/526095
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Cherry Mui <cherryyz@google.com>
src/cmd/go/internal/gover/toolchain.go
src/cmd/go/testdata/script/mod_toolchain_slash.txt [new file with mode: 0644]