2009-10-22 Paul Pluzhnikov <ppluzhnikov@google.com>

	PR gdb/10819
	* dwarf2-frame.c (find_cie): Don't call bsearch on empty cie_table.
	* objfiles.c (find_pc_section): Likewise.
	(update_section_map): Don't allocate empty table.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 2951bae2e4c64fe29f8ec1ab0eab3efc00e77147..f30822cc4cd92b38e22d8006f6b55407d9a8c1d3 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+2009-10-22  Paul Pluzhnikov  <ppluzhnikov@google.com>
+
+       PR gdb/10819
+       * dwarf2-frame.c (find_cie): Don't call bsearch on empty cie_table.
+       * objfiles.c (find_pc_section): Likewise.
+       (update_section_map): Don't allocate empty table.
+       
 2009-10-16  Hui Zhu  <teawater@gmail.com>
            Michael Snyder  <msnyder@msnyder-server.eng.vmware.com>
 

diff --git a/gdb/dwarf2-frame.c b/gdb/dwarf2-frame.c
index 9c464ba7e94697f36e99ff1d679bd43970cef659..5e9be2ccb2cc7ae896725ade66755e5c79672362 100644 (file)
--- a/gdb/dwarf2-frame.c
+++ b/gdb/dwarf2-frame.c
@@ -1525,6 +1525,14 @@ find_cie (struct dwarf2_cie_table *cie_table, ULONGEST cie_pointer)
 {
   struct dwarf2_cie **p_cie;
 
+  /* The C standard (ISO/IEC 9899:TC2) requires the BASE argument to
+     bsearch be non-NULL.  */
+  if (cie_table->entries == NULL)
+    {
+      gdb_assert (cie_table->num_entries == 0);
+      return NULL;
+    }
+
   p_cie = bsearch (&cie_pointer, cie_table->entries, cie_table->num_entries,
                    sizeof (cie_table->entries[0]), bsearch_cie_cmp);
   if (p_cie != NULL)

diff --git a/gdb/objfiles.c b/gdb/objfiles.c
index 13a8c555d7607f959ae6decbe252ac63771cdd74..03a49a935e9bcc9c2f251e348f61c33330e26a3b 100644 (file)
--- a/gdb/objfiles.c
+++ b/gdb/objfiles.c
@@ -1113,6 +1113,14 @@ update_section_map (struct program_space *pspace,
       if (insert_section_p (objfile->obfd, s->the_bfd_section))
        alloc_size += 1;
 
+  /* This happens on detach/attach (e.g. in gdb.base/attach.exp).  */
+  if (alloc_size == 0)
+    {
+      *pmap = NULL;
+      *pmap_size = 0;
+      return;
+    }
+
   map = xmalloc (alloc_size * sizeof (*map));
 
   i = 0;
@@ -1175,6 +1183,14 @@ find_pc_section (CORE_ADDR pc)
       pspace_info->objfiles_changed_p = 0;
     }
 
+  /* The C standard (ISO/IEC 9899:TC2) requires the BASE argument to
+     bsearch be non-NULL.  */
+  if (pspace_info->sections == NULL)
+    {
+      gdb_assert (pspace_info->num_sections == 0);
+      return NULL;
+    }
+
   sp = (struct obj_section **) bsearch (&pc,
                                        pspace_info->sections,
                                        pspace_info->num_sections,