Blame SOURCES/gdb-archer-pie-addons-keep-disabled.patch

0b3064
From FEDORA_PATCHES Mon Sep 17 00:00:00 2001
0b3064
From: Fedora GDB patches <invalid@email.com>
0b3064
Date: Fri, 27 Oct 2017 21:07:50 +0200
0b3064
Subject: gdb-archer-pie-addons-keep-disabled.patch
0b3064
0b3064
;;=push+jan: Breakpoints disabling matching should not be based on address.
0b3064
0b3064
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
0b3064
--- a/gdb/breakpoint.c
0b3064
+++ b/gdb/breakpoint.c
0b3064
@@ -15396,6 +15396,51 @@ static struct cmd_list_element *enablebreaklist = NULL;
0b3064
 
0b3064
 cmd_list_element *commands_cmd_element = nullptr;
0b3064
 
0b3064
+void
0b3064
+breakpoints_relocate (struct objfile *objfile, struct section_offsets *delta)
0b3064
+{
0b3064
+  struct bp_location *bl, **blp_tmp;
0b3064
+  int changed = 0;
0b3064
+
0b3064
+  gdb_assert (objfile->separate_debug_objfile_backlink == NULL);
0b3064
+
0b3064
+  ALL_BP_LOCATIONS (bl, blp_tmp)
0b3064
+    {
0b3064
+      struct obj_section *osect;
0b3064
+
0b3064
+      /* BL->SECTION can be correctly NULL for breakpoints with multiple
0b3064
+         locations expanded through symtab.  */
0b3064
+
0b3064
+      ALL_OBJFILE_OSECTIONS (objfile, osect)
0b3064
+	{
0b3064
+	  CORE_ADDR relocated_address;
0b3064
+	  CORE_ADDR delta_offset;
0b3064
+
0b3064
+	  delta_offset = ANOFFSET (delta, osect->the_bfd_section->index);
0b3064
+	  if (delta_offset == 0)
0b3064
+	    continue;
0b3064
+	  relocated_address = bl->address + delta_offset;
0b3064
+
0b3064
+	  if (obj_section_addr (osect) <= relocated_address
0b3064
+	      && relocated_address < obj_section_endaddr (osect))
0b3064
+	    {
0b3064
+	      if (bl->inserted)
0b3064
+		remove_breakpoint (bl);
0b3064
+
0b3064
+	      bl->address += delta_offset;
0b3064
+	      bl->requested_address += delta_offset;
0b3064
+
0b3064
+	      changed = 1;
0b3064
+	    }
0b3064
+	}
0b3064
+    }
0b3064
+
0b3064
+  if (changed)
0b3064
+    std::sort (bp_locations, bp_locations + bp_locations_count,
0b3064
+	       bp_location_is_less_than);
0b3064
+}
0b3064
+
0b3064
+void _initialize_breakpoint (void);
0b3064
 void
0b3064
 _initialize_breakpoint (void)
0b3064
 {
0b3064
diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h
0b3064
--- a/gdb/breakpoint.h
0b3064
+++ b/gdb/breakpoint.h
0b3064
@@ -1696,6 +1696,9 @@ extern const char *ep_parse_optional_if_clause (const char **arg);
0b3064
    UIOUT iff debugging multiple threads.  */
0b3064
 extern void maybe_print_thread_hit_breakpoint (struct ui_out *uiout);
0b3064
 
0b3064
+extern void breakpoints_relocate (struct objfile *objfile,
0b3064
+				  struct section_offsets *delta);
0b3064
+
0b3064
 /* Print the specified breakpoint.  */
0b3064
 extern void print_breakpoint (breakpoint *bp);
0b3064
 
0b3064
diff --git a/gdb/objfiles.c b/gdb/objfiles.c
0b3064
--- a/gdb/objfiles.c
0b3064
+++ b/gdb/objfiles.c
0b3064
@@ -816,6 +816,11 @@ objfile_relocate1 (struct objfile *objfile,
0b3064
 				obj_section_addr (s));
0b3064
     }
0b3064
 
0b3064
+  /* Final call of breakpoint_re_set can keep breakpoint locations disabled if
0b3064
+     their addresses match.  */
0b3064
+  if (objfile->separate_debug_objfile_backlink == NULL)
0b3064
+    breakpoints_relocate (objfile, delta);
0b3064
+
0b3064
   /* Data changed.  */
0b3064
   return 1;
0b3064
 }
0b3064
diff --git a/gdb/value.c b/gdb/value.c
0b3064
--- a/gdb/value.c
0b3064
+++ b/gdb/value.c
0b3064
@@ -2840,7 +2840,7 @@ value_static_field (struct type *type, int fieldno)
0b3064
     case FIELD_LOC_KIND_PHYSADDR:
0b3064
       retval = value_at_lazy (TYPE_FIELD_TYPE (type, fieldno),
0b3064
 			      TYPE_FIELD_STATIC_PHYSADDR (type, fieldno)
0b3064
-			      + (TYPE_OBJFILE (type) == NULL ? 0 : TYPE_OBJFILE (type)->section_offsets[SECT_OFF_TEXT (TYPE_OBJFILE (type))]));
0b3064
+			      + (TYPE_OBJFILE (type) == NULL ? 0 : ANOFFSET (TYPE_OBJFILE (type)->section_offsets, SECT_OFF_TEXT (TYPE_OBJFILE (type)))));
0b3064
       break;
0b3064
     case FIELD_LOC_KIND_PHYSNAME:
0b3064
     {