This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
[PATCH] Fix ppc64 .note.GNU-stack handling
- From: Jakub Jelinek <jakub at redhat dot com>
- To: binutils at sources dot redhat dot com
- Date: Wed, 9 Feb 2005 20:02:18 +0100
- Subject: [PATCH] Fix ppc64 .note.GNU-stack handling
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
Hi!
.note.GNU-stack section handling is broken on ppc64 (and probably on hppa as
well). If there are any .note.GNU-stack sections in the input, then
all of them must have that section and without SEC_CODE, otherwise the
result is executable. But on ppc64 there is a linker stubs magic bfd
synthetized by the linker that never has this section, so if there are
any .note.GNU-stack sections, whether executable or not, the result
is PT_GNU_STACK RWE.
Fixed thusly, ok to commit?
2005-02-09 Jakub Jelinek <jakub@redhat.com>
* bfd-in.h (BFD_LINKER_CREATED): Define.
* bfd-in2.h: Rebuilt.
* elflink.c (bfd_elf_size_dynamic_sections): Disregard
BFD_LINKER_CREATED bfds when determining input bfds without
.note.GNU-stack section.
* emultempl/hppaelf.em (hppaelf_create_output_section_statements):
Set BFD_LINKER_CREATED on the stubs bfd.
* emultempl/ppc64elf.em (ppc_create_output_section_statements):
Likewise.
--- bfd/bfd-in.h.jj 2005-02-09 16:06:50.000000000 +0100
+++ bfd/bfd-in.h 2005-02-09 17:44:34.000000000 +0100
@@ -1,7 +1,7 @@
/* Main header file for the bfd library -- portable access to object files.
Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
- 1999, 2000, 2001, 2002, 2003, 2004 Free Software Foundation, Inc.
+ 1999, 2000, 2001, 2002, 2003, 2004, 2005 Free Software Foundation, Inc.
Contributed by Cygnus Support.
@@ -230,6 +230,10 @@ bfd_format;
/* The sections in this BFD specify a memory page. */
#define HAS_LOAD_PAGE 0x1000
+
+/* This BFD has been created by the linker and doesn't correspond
+ to any input file. */
+#define BFD_LINKER_CREATED 0x2000
/* Symbols and relocation. */
--- bfd/bfd-in2.h.jj 2005-02-09 16:06:50.000000000 +0100
+++ bfd/bfd-in2.h 2005-02-09 17:45:09.698719811 +0100
@@ -8,7 +8,7 @@
/* Main header file for the bfd library -- portable access to object files.
Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
- 1999, 2000, 2001, 2002, 2003, 2004 Free Software Foundation, Inc.
+ 1999, 2000, 2001, 2002, 2003, 2004, 2005 Free Software Foundation, Inc.
Contributed by Cygnus Support.
@@ -245,6 +245,10 @@ bfd_format;
/* The sections in this BFD specify a memory page. */
#define HAS_LOAD_PAGE 0x1000
+
+/* This BFD has been created by the linker and doesn't correspond
+ to any input file. */
+#define BFD_LINKER_CREATED 0x2000
/* Symbols and relocation. */
--- bfd/elflink.c.jj 2005-02-09 16:06:57.000000000 +0100
+++ bfd/elflink.c 2005-02-09 17:47:45.209950411 +0100
@@ -4895,7 +4895,7 @@ bfd_elf_size_dynamic_sections (bfd *outp
{
asection *s;
- if (inputobj->flags & DYNAMIC)
+ if (inputobj->flags & (DYNAMIC | BFD_LINKER_CREATED))
continue;
s = bfd_get_section_by_name (inputobj, ".note.GNU-stack");
if (s)
--- ld/emultempl/hppaelf.em.jj 2005-01-21 13:01:18.000000000 +0100
+++ ld/emultempl/hppaelf.em 2005-02-09 17:46:46.442444375 +0100
@@ -88,6 +88,7 @@ hppaelf_create_output_section_statements
return;
}
+ stub_file->the_bfd->flags |= BFD_LINKER_CREATED;
ldlang_add_file (stub_file);
}
--- ld/emultempl/ppc64elf.em.jj 2005-01-20 09:50:47.000000000 +0100
+++ ld/emultempl/ppc64elf.em 2005-02-09 17:47:11.103040786 +0100
@@ -92,6 +92,7 @@ ppc_create_output_section_statements (vo
return;
}
+ stub_file->the_bfd->flags |= BFD_LINKER_CREATED;
ldlang_add_file (stub_file);
ppc64_elf_init_stub_bfd (stub_file->the_bfd, &link_info);
}
Jakub