* NEWS: Document that "info symbol" works with COFF debug info and

	its variants.

	* minsyms.c (lookup_minimal_symbol_by_pc_section): Don't skip
	symbols whose SYMBOL_BFD_SECTION is NULL.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index ae38ff0..09b69a6 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,11 @@
+2001-02-11  Eli Zaretskii  <eliz@is.elta.co.il>
+
+       * NEWS: Document that "info symbol" works with COFF debug info and
+       its variants.
+
+       * minsyms.c (lookup_minimal_symbol_by_pc_section): Don't skip
+       symbols whose SYMBOL_BFD_SECTION is NULL.
+
 2000-02-10  Peter Schauer  <pes@regent.e-technik.tu-muenchen.de>
 
        Get rid of AIX specific PC_LOAD_SEGMENT, replace with PC_SOLIB.

diff --git a/gdb/NEWS b/gdb/NEWS
index e4bd55a..dc6ad4c 100644 (file)
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -3,6 +3,8 @@
 
 *** Changes since GDB 5.0:
 
+* "info symbol" works on platforms which use COFF, ECOFF, XCOFF, and NLM.
+
 * New native configurations
 
 Alpha FreeBSD                                  alpha*-*-freebsd*

diff --git a/gdb/minsyms.c b/gdb/minsyms.c
index 9070b6f..62cd00f 100644 (file)
--- a/gdb/minsyms.c
+++ b/gdb/minsyms.c
@@ -482,6 +482,10 @@ lookup_minimal_symbol_by_pc_section (CORE_ADDR pc, asection *section)
              /* This is the new code that distinguishes it from the old function */
              if (section)
                while (hi >= 0
+                      /* Some types of debug info, such as COFF,
+                         don't fill the bfd_section member, so don't
+                         throw away symbols on those platforms.  */
+                      && SYMBOL_BFD_SECTION (&msymbol[hi]) != NULL
                       && SYMBOL_BFD_SECTION (&msymbol[hi]) != section)
                  --hi;