Return zero only as the last step of qsort function.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index f451dde..71208e5 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,7 @@
+2001-08-01  Adam Nemet  <anemet@lnxw.com>
+
+       * elf.c (elf_sort_sections): Return zero only as the last step.
+
 2001-08-01  Nick Clifton  <nickc@cambridge.redhat.com>
 
        * config.bfd (arm-vxworks): Change name of define from VXWORKS to

diff --git a/bfd/elf.c b/bfd/elf.c
index 4e2d7ae..8dc2f31 100644 (file)
--- a/bfd/elf.c
+++ b/bfd/elf.c
@@ -2677,18 +2677,22 @@ elf_sort_sections (arg1, arg2)
   if (TOEND (sec1))
     {
       if (TOEND (sec2))
-       return sec1->target_index - sec2->target_index;
+       {
+         /* If the indicies are the same, do not return 0
+            here, but continue to try the next comparison.  */
+         if (sec1->target_index - sec2->target_index != 0)
+           return sec1->target_index - sec2->target_index;
+       }
       else
        return 1;
     }
-
-  if (TOEND (sec2))
+  else if (TOEND (sec2))
     return -1;
 
 #undef TOEND
 
-  /* Sort by size, to put zero sized sections before others at the
-     same address.  */
+  /* Sort by size, to put zero sized sections
+     before others at the same address.  */
 
   if (sec1->_raw_size < sec2->_raw_size)
     return -1;