• Russ Cox's avatar
    cmd/go: accept only limited compiler and linker flags in #cgo directives · 1dcb5836
    Russ Cox authored
    Both gcc and clang accept an option -fplugin=code.so to load
    a plugin from the ELF shared object file code.so.
    Obviously that plugin can then do anything it wants
    during the build. This is contrary to the goal of "go get"
    never running untrusted code during the build.
    (What happens if you choose to run the result of
    the build is your responsibility.)
    
    Disallow this behavior by only allowing a small set of
    known command-line flags in #cgo CFLAGS directives
    (and #cgo LDFLAGS, etc).
    
    The new restrictions can be adjusted by the environment
    variables CGO_CFLAGS_ALLOW, CGO_CFLAGS_DISALLOW,
    and so on. See the documentation.
    
    In addition to excluding cgo-defined flags, we also have to
    make sure that when we pass file names on the command
    line, they don't look like flags. So we now refuse to build
    packages containing suspicious file names like -x.go.
    
    A wrinkle in all this is that GNU binutils uniformly accept
    @foo on the command line to mean "if the file foo exists,
    then substitute its contents for @foo in the command line".
    So we must also reject @x.go, flags and flag arguments
    beginning with @, and so on.
    
    Fixes #23672, CVE-2018-6574.
    
    Change-Id: I59e7c1355155c335a5c5ae0d2cf8fa7aa313940a
    Reviewed-on: https://team-review.git.corp.google.com/209949Reviewed-by: 's avatarIan Lance Taylor <iant@google.com>
    1dcb5836
Name
Last commit
Last update
..
archive Loading commit data...
bufio Loading commit data...
builtin Loading commit data...
bytes Loading commit data...
cmd Loading commit data...
compress Loading commit data...
container Loading commit data...
context Loading commit data...
crypto Loading commit data...
database/sql Loading commit data...
debug Loading commit data...
encoding Loading commit data...
errors Loading commit data...
expvar Loading commit data...
flag Loading commit data...
fmt Loading commit data...
go Loading commit data...
hash Loading commit data...
html Loading commit data...
image Loading commit data...
index/suffixarray Loading commit data...
internal Loading commit data...
io Loading commit data...
log Loading commit data...
math Loading commit data...
mime Loading commit data...
net Loading commit data...
os Loading commit data...
path Loading commit data...
plugin Loading commit data...
reflect Loading commit data...
regexp Loading commit data...
runtime Loading commit data...
sort Loading commit data...
strconv Loading commit data...
strings Loading commit data...
sync Loading commit data...
syscall Loading commit data...
testing Loading commit data...
text Loading commit data...
time Loading commit data...
unicode Loading commit data...
unsafe Loading commit data...
vendor/golang_org/x Loading commit data...
Make.dist Loading commit data...
all.bash Loading commit data...
all.bat Loading commit data...
all.rc Loading commit data...
androidtest.bash Loading commit data...
bootstrap.bash Loading commit data...
buildall.bash Loading commit data...
clean.bash Loading commit data...
clean.bat Loading commit data...
clean.rc Loading commit data...
cmp.bash Loading commit data...
iostest.bash Loading commit data...
make.bash Loading commit data...
make.bat Loading commit data...
make.rc Loading commit data...
naclmake.bash Loading commit data...
nacltest.bash Loading commit data...
race.bash Loading commit data...
race.bat Loading commit data...
run.bash Loading commit data...
run.bat Loading commit data...
run.rc Loading commit data...