This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [RFA] Enhance stabs reader to better deal with forward references
- From: "Pierre Muller" <muller at ics dot u-strasbg dot fr>
- To: "'Joel Brobecker'" <brobecker at adacore dot com>, "'Pedro Alves'" <pedro_alves at portugalmail dot pt>
- Cc: <gdb-patches at sourceware dot org>
- Date: Mon, 2 Apr 2007 10:51:28 +0200
- Subject: RE: [RFA] Enhance stabs reader to better deal with forward references
- References: <20070209194107.GA3365@adacore.com> <20070227165003.GB31729@caradoc.them.org> <20070228201201.GH13140@adacore.com> <20070327162947.GC3452@caradoc.them.org> <20070329183609.GA3737@adacore.com> <460EC794.2080309@portugalmail.pt> <20070402070011.GC3963@adacore.com>
I made the same change to my local repository,
due to the same problem, on the same target.
The problem is that in the CVS tree, end_symtab function
is called from within the read_ofile_symtab function
after current_objfile is reset to nil.
But due to your change, current_objfile is still
accessed inside dbx_lookup_type (stabsread.c)
called by cleanup_undefined_types_noname (stabsread.c)
called by cleanup_undefined_types (stabsread.c)
called by end_symtab (buildsym.c).
Hope this helps,
Pierre
-----Message d'origine-----
De?: gdb-patches-owner@sourceware.org
[mailto:gdb-patches-owner@sourceware.org] De la part de Joel Brobecker
Envoyé?: Monday, April 02, 2007 9:00 AM
À?: Pedro Alves
Cc?: gdb-patches@sourceware.org
Objet?: Re: [RFA] Enhance stabs reader to better deal with forward
references
Pedro,
> Unfortunatelly, this patch broke Cygwin, which still uses stabs by
default.
>
> I am getting a SIGSEGV while issuing a breakpoint.
Thanks for the report, and sorry about that. Unfortunately, I won't have
much time in the next two or three days - I might be able to understand
what is actually from the backtrace and your patch, but it might not be
sufficient. And if you have an opportunity to investigate a bit more,
that would be very much appreciated. In the meantime, feel free to back
the patch out if necessary.
--
Joel