diff options
| author | Jan Kratochvil <jan.kratochvil@redhat.com> | 2010-01-08 08:55:15 +0000 |
|---|---|---|
| committer | Jan Kratochvil <jan.kratochvil@redhat.com> | 2010-01-08 08:55:15 +0000 |
| commit | 8212bf0441f42f6bf7952d4caa0f2b2ae8b3381e (patch) | |
| tree | 997284d99752603e3c472eff62c52193e7edb0a6 /gdb | |
| parent | 915d481cfb685dfe9f4f17f58e6da3b0a87e0d86 (diff) | |
| download | gdb-8212bf0441f42f6bf7952d4caa0f2b2ae8b3381e.tar.gz | |
gdb/
* stabsread.c (read_args): Handle zero arguments.
gdb/testsuite/
* gdb.stabs/weird.def (args93): New.
Diffstat (limited to 'gdb')
| -rw-r--r-- | gdb/ChangeLog | 4 | ||||
| -rw-r--r-- | gdb/stabsread.c | 12 | ||||
| -rw-r--r-- | gdb/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gdb/testsuite/gdb.stabs/weird.def | 4 |
4 files changed, 23 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1657c7ddcbc..9cc7d5beacb 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2010-01-09 Jan Kratochvil <jan.kratochvil@redhat.com> + + * stabsread.c (read_args): Handle zero arguments. + 2009-01-08 Joel Brobecker <brobecker@adacore.com> Cannot find in-tree libiconv.a after reconfigure. diff --git a/gdb/stabsread.c b/gdb/stabsread.c index 66b084ffc66..71c168c450a 100644 --- a/gdb/stabsread.c +++ b/gdb/stabsread.c @@ -4111,7 +4111,17 @@ read_args (char **pp, int end, struct objfile *objfile, int *nargsp, } (*pp)++; /* get past `end' (the ':' character) */ - if (TYPE_CODE (types[n - 1]) != TYPE_CODE_VOID) + if (n == 0) + { + /* We should read at least the THIS parameter here. Some broken stabs + output contained `(0,41),(0,42)=@s8;-16;,(0,43),(0,1);' where should + have been present ";-16,(0,43)" reference instead. This way the + excessive ";" marker prematurely stops the parameters parsing. */ + + complaint (&symfile_complaints, _("Invalid (empty) method arguments")); + *varargsp = 0; + } + else if (TYPE_CODE (types[n - 1]) != TYPE_CODE_VOID) *varargsp = 1; else { diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 208ffa5c25c..3b0c421a039 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2010-01-09 Jan Kratochvil <jan.kratochvil@redhat.com> + + * gdb.stabs/weird.def (args93): New. + 2010-01-07 Doug Evans <dje@google.com> * lib/gdb.exp (gdb_skip_xml_test): Add comment. diff --git a/gdb/testsuite/gdb.stabs/weird.def b/gdb/testsuite/gdb.stabs/weird.def index 33116b4c5f3..71fcff677ff 100644 --- a/gdb/testsuite/gdb.stabs/weird.def +++ b/gdb/testsuite/gdb.stabs/weird.def @@ -880,3 +880,7 @@ var3: .stabs "sym92:\ !#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[]^_`abcdefghijklmnopqrstuvwxyz{|}~",N_LSYM,0,0,0 .stabs "type92:t92=\ !#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[]^_`abcdefghijklmnopqrstuvwxyz{|}~",N_LSYM,0,0,0 .stabs "attr92:G392=@\ !#$%&'()*+,-./0123456789:<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[]^_`abcdefghijklmnopqrstuvwxyz{|}~;1",N_GSYM,0,0, 0 + +# See read_args "Invalid (empty) method arguments" error; there is an +# unexpected semi-colon after =@s8;-16 that used to cause a GDB crash. +.stabs "args93:G93=#(0,93),(0,93)=@s8;-16;,(0,93),(0,93);",N_GSYM,0,0,0 |