From e8afb6d87f76668ce4b4b560d89e4ecc2b7d974f Mon Sep 17 00:00:00 2001
From: "Devon H. O'Dell" <devon.odell@gmail.com>
Date: Wed, 13 Jan 2010 16:48:14 -0800
Subject: [PATCH] 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
---
 src/cmd/cgo/gcc.go | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/cmd/cgo/gcc.go b/src/cmd/cgo/gcc.go
index 4f65a1afb7..c525b492aa 100644
--- a/src/cmd/cgo/gcc.go
+++ b/src/cmd/cgo/gcc.go
@@ -48,7 +48,13 @@ func (p *Prog) loadDebugInfo() {
 			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.