cgo: Only allow numeric / string / character type constants for references
to #defined things.
Fixes #520.
R=rsc, rsaarelm
CC=golang-dev
https://golang.org/cl/186138
diff --git a/src/cmd/cgo/gcc.go b/src/cmd/cgo/gcc.go
index 4f65a1a..c525b49 100644
--- a/src/cmd/cgo/gcc.go
+++ b/src/cmd/cgo/gcc.go
@@ -48,7 +48,13 @@
val = strings.TrimSpace(line[tabIndex:])
}
- defines[key] = val
+ // Only allow string, character, and numeric constants. Ignoring #defines for
+ // symbols allows those symbols to be referenced in Go, as they will be
+ // translated by gcc later.
+ _, err := strconv.Atoi(string(val[0]))
+ if err == nil || val[0] == '\'' || val[0] == '"' {
+ defines[key] = val
+ }
}
// Construct a slice of unique names from p.Crefs.