]> Cypherpunks.ru repositories - gostls13.git/commit
cmd/go: don't lookup the path for CC when invoking cgo
authorJay Conrod <jayconrod@google.com>
Fri, 22 Jan 2021 19:27:24 +0000 (14:27 -0500)
committerJay Conrod <jayconrod@google.com>
Fri, 22 Jan 2021 21:16:10 +0000 (21:16 +0000)
commita2cef9b544708ecae983ed8836ee2425a28aab68
tree80ebd937e34ba17a9e66005f5eadf9ee441f6bc6
parentdab3e5affefb7e0b157ad27fe4797c4c6f3c4ea6
cmd/go: don't lookup the path for CC when invoking cgo

Previously, if CC was a path without separators (like gcc or clang),
we'd look it up in PATH in cmd/go using internal/execabs.LookPath,
then pass the resolved path to cgo in CC.

This caused a regression: if the directory in PATH containing CC has a
space, cgo splits it and interprets it as multiple arguments.

With this change, cmd/go no longer resolves CC before invoking
cgo. cgo does the path lookup on each invocation. This reverts the
security fix CL 284780, but that was redundant with the addition of
internal/execabs (CL 955304), which still protects us.

Fixes #43808
Updates #41400

Change-Id: I65d91a1e303856df8653881eb6e2e75a3bf95c49
Reviewed-on: https://go-review.googlesource.com/c/go/+/285873
Trust: Jay Conrod <jayconrod@google.com>
Run-TryBot: Jay Conrod <jayconrod@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
src/cmd/go/internal/work/action.go
src/cmd/go/internal/work/exec.go
src/cmd/go/testdata/script/cgo_path.txt
src/cmd/go/testdata/script/cgo_path_space.txt [new file with mode: 0644]