This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] Fix for searching for data symbols


Daniel Jacobowitz wrote:
> 
> On Sat, Jan 19, 2002 at 09:52:22AM -0800, Michael Snyder wrote:
> > Daniel Jacobowitz wrote:
> > >
> > > This patch fixes the problem I described in:
> > >         http://sources.redhat.com/ml/gdb/2002-01/msg00223.html
> > >
> > > Basically, since we use the bounds of the text segment to search for symbols
> > > in find_pc_*, don't let them return a bogus answer for a symbol we know is
> > > off in data land.
> > >
> > > OK to commit, or does anyone prefer a different approach to this?
> >
> > Is there any chance that a data symbol that we don't want
> > would shadow a text symbol that we do want?  Eg. maybe in
> > one module we have
> >       static int foo;
> > and in another we have
> >       extern int foo(void);
> > Maybe they are even in separate objfiles (shared libs).
> > Maybe lookup_minimal_symbol will find the wrong one first,
> > and your change will terminate the search before we would
> > have found the right one?
> >
> > Just speculating...
> 
> Well, we aren't using lookup_minimal_symbol; we're using
> lookup_minimal_symbol_by_pc_section.  I don't think it's possible for
> it to get the wrong one, since it's by address instead of by name.

OK, good, that answers that concern.  Thanks.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]