aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoland McGrath <mcgrathr@google.com>2020-01-27 11:14:33 -0800
committerRoland McGrath <mcgrathr@google.com>2020-01-27 11:15:01 -0800
commit3828695c134afca84c30cdd333a68c7f1529c1cf (patch)
tree80742ded623af7e4d2b926a2c2a85ce159b87cff
parent2cd7b00eb1337e427fa6149779157d8638eb57d9 (diff)
downloadgdb-users/roland/2.34/objcopy-merge-notes-qsort.zip
gdb-users/roland/2.34/objcopy-merge-notes-qsort.tar.gz
gdb-users/roland/2.34/objcopy-merge-notes-qsort.tar.bz2
Fix objcopy --merge-notes dependency on qsort implementation behavior.users/roland/2.34/objcopy-merge-notes-qsort
binutils/ * objcopy.c (compare_gnu_build_notes): Fix comparison results for overlapping ranges so that (A == B) == (B == A) holds. (cherry picked from commit 82ef9cad78b74ee9176e059e8c8dec1bdb4a1efe)
-rw-r--r--binutils/ChangeLog5
-rw-r--r--binutils/objcopy.c34
2 files changed, 23 insertions, 16 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 3005cdf..dc95ee3 100644
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,8 @@
+2020-01-27 Roland McGrath <mcgrathr@google.com>
+
+ * objcopy.c (compare_gnu_build_notes): Fix comparison results
+ for overlapping ranges so that (A == B) == (B == A) holds.
+
2020-01-23 Nick Clifton <nickc@redhat.com>
* po/fr.po: Updated French translation.
diff --git a/binutils/objcopy.c b/binutils/objcopy.c
index ef3b693..212e251 100644
--- a/binutils/objcopy.c
+++ b/binutils/objcopy.c
@@ -1945,7 +1945,7 @@ typedef struct objcopy_internal_note
bfd_vma start;
bfd_vma end;
} objcopy_internal_note;
-
+
#define DEBUG_MERGE 0
#if DEBUG_MERGE
@@ -2041,7 +2041,7 @@ compare_gnu_build_notes (const void * data1, const void * data2)
pnote1->note.namesz - 3 : pnote2->note.namesz - 3);
if (cmp)
return cmp;
-
+
if (pnote1->end < pnote2->start)
return -1;
if (pnote1->start > pnote2->end)
@@ -2052,13 +2052,15 @@ compare_gnu_build_notes (const void * data1, const void * data2)
return -1;
if (pnote1->end > pnote2->end)
return 1;
-
+ if (pnote1->end < pnote2->end)
+ return -1;
+
/* Put OPEN notes before function notes. */
if (is_open_note (pnote1) && ! is_open_note (pnote2))
return -1;
if (! is_open_note (pnote1) && is_open_note (pnote2))
return 1;
-
+
return 0;
}
@@ -2086,7 +2088,7 @@ sort_gnu_build_notes (const void * data1, const void * data2)
return 1; /* 1: F 2: O */
}
-
+
/* Sort by starting address. */
if (pnote1->start < pnote2->start)
return -1;
@@ -2104,7 +2106,7 @@ sort_gnu_build_notes (const void * data1, const void * data2)
&& pnote2->note.namesz > 4
&& pnote1->note.namedata[3] != pnote2->note.namedata[3])
return pnote1->note.namedata[3] - pnote2->note.namedata[3];
-
+
return 0;
}
@@ -2151,7 +2153,7 @@ merge_gnu_build_notes (bfd * abfd,
goto done;
}
}
-
+
/* Make a copy of the notes and convert to our internal format.
Minimum size of a note is 12 bytes. Also locate the version
notes and check them. */
@@ -2232,7 +2234,7 @@ merge_gnu_build_notes (bfd * abfd,
address. */
end = (bfd_vma) -1;
break;
-
+
case 8:
if (! is_64bit (abfd))
{
@@ -2257,7 +2259,7 @@ merge_gnu_build_notes (bfd * abfd,
start = bfd_get_64 (abfd, pnote->note.descdata);
end = bfd_get_64 (abfd, pnote->note.descdata + 8);
break;
-
+
default:
err = _("corrupt GNU build attribute note: bad description size");
goto done;
@@ -2396,7 +2398,7 @@ merge_gnu_build_notes (bfd * abfd,
|| memcmp (back->note.namedata,
pnote->note.namedata, pnote->note.namesz) != 0)
break;
-
+
if (back->start == pnote->start
&& back->end == pnote->end)
{
@@ -2450,7 +2452,7 @@ merge_gnu_build_notes (bfd * abfd,
if (! is_deleted_note (pnote))
merge_debug ("Unable to do anything with note at %#08lx\n",
(pnote->note.namedata - (char *) contents) - 12);
-#endif
+#endif
}
/* Resort the notes. */
@@ -2532,7 +2534,7 @@ merge_gnu_build_notes (bfd * abfd,
pnote->note.namesz
);
#endif
-
+
new_size = new - new_contents;
if (new_size < size)
{
@@ -4057,7 +4059,7 @@ setup_section (bfd *ibfd, sec_ptr isection, void *obfdarg)
alignment = p->alignment;
else
alignment = bfd_section_alignment (isection);
-
+
/* FIXME: This is probably not enough. If we change the LMA we
may have to recompute the header for the file as well. */
if (!bfd_set_section_alignment (osection, alignment))
@@ -5468,7 +5470,7 @@ copy_main (int argc, char *argv[])
s = strchr (optarg, '=');
if (s == NULL)
fatal (_("bad format for --set-section-alignment: argument needed"));
-
+
align = atoi (s + 1);
if (align <= 0)
fatal (_("bad format for --set-section-alignment: numeric argument needed"));
@@ -5480,7 +5482,7 @@ copy_main (int argc, char *argv[])
align >>= 1;
++palign;
}
-
+
if (align != 1)
/* Number has more than on 1, i.e. wasn't a power of 2. */
fatal (_("bad format for --set-section-alignment: alignment is not a power of two"));
@@ -5496,7 +5498,7 @@ copy_main (int argc, char *argv[])
p->alignment = palign;
}
break;
-
+
case OPTION_RENAME_SECTION:
{
flagword flags;