diff options
| author | Anthony Martin <ality@pbrane.org> | 2013-04-03 18:23:43 -0700 |
|---|---|---|
| committer | Anthony Martin <ality@pbrane.org> | 2013-04-03 18:23:43 -0700 |
| commit | 272687ec24ca828f8c7ba22c64e0060436124051 (patch) | |
| tree | 34e25f300178cca2f1df434df57682ab3795c669 /src/cmd/nm/nm.c | |
| parent | 1b9e36356be410adf3b0b80e99996f8f3476d8cf (diff) | |
| download | go-272687ec24ca828f8c7ba22c64e0060436124051.tar.xz | |
cmd/nm: don't add filename elements for m symbols
The compilers used to generate only one 'm' symbol
to record the stack frame size for each function.
In cmd/nm, the 'm' and 'f' symbols are handled in
the same switch case with a special exception for
the symbol described above called ".frame".
Now that the compilers emit additional 'm' symbols
for precise garbage collection of the stack, the
current logic is incorrect. cmd/nm will attempt to
interpret these new 'm' symbols as 'f' symbols and
add them to the file name index table.
This fails with an out-of-memory condition when
zenter encounters an 'm' symbol with a very large
value (usually the .args symbol indicating a
variadic NOSPLIT function).
R=iant
CC=dave, gobot, golang-dev, rsc
https://golang.org/cl/7962045
Diffstat (limited to 'src/cmd/nm/nm.c')
| -rw-r--r-- | src/cmd/nm/nm.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/cmd/nm/nm.c b/src/cmd/nm/nm.c index 6682390358..aa056b8825 100644 --- a/src/cmd/nm/nm.c +++ b/src/cmd/nm/nm.c @@ -275,11 +275,13 @@ psym(Sym *s, void* p) return; break; case 'm': + if(!aflag || uflag || gflag) + return; + break; case 'f': /* we only see a 'z' when the following is true*/ if(!aflag || uflag || gflag) return; - if (strcmp(s->name, ".frame")) - zenter(s); + zenter(s); break; case 'a': case 'p': |