public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* PATCH: PR ld/12982: BFD linker LTO plugin makes stack executable by defaul
@ 2011-07-11 19:49 H.J. Lu
  0 siblings, 0 replies; only message in thread
From: H.J. Lu @ 2011-07-11 19:49 UTC (permalink / raw)
  To: binutils

Hi,

We shouldn't check BFD_PLUGIN for GNU_STACK.  I checked it in as an
obvious fix.

Thanks.

H.J.
---
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index e20d3db..f10bef2 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2011-07-11  H.J. Lu  <hongjiu.lu@intel.com>
+
+	PR ld/12982
+	* elflink.c (bfd_elf_size_dynamic_sections): Also skip BFD_PLUGIN
+	when setting stack_flags.
+
 2011-07-11  Catherine Moore  <clm@codesourcery.com>
 
 	* aout-adobe.c (aout_32_bfd_lookup_section_flags): New definition.
diff --git a/bfd/elflink.c b/bfd/elflink.c
index e2cb28c..dc3840e 100644
--- a/bfd/elflink.c
+++ b/bfd/elflink.c
@@ -5525,7 +5525,8 @@ bfd_elf_size_dynamic_sections (bfd *output_bfd,
 	{
 	  asection *s;
 
-	  if (inputobj->flags & (DYNAMIC | EXEC_P | BFD_LINKER_CREATED))
+	  if (inputobj->flags
+	      & (DYNAMIC | EXEC_P | BFD_PLUGIN | BFD_LINKER_CREATED))
 	    continue;
 	  s = bfd_get_section_by_name (inputobj, ".note.GNU-stack");
 	  if (s)

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-07-11 18:40 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-11 19:49 PATCH: PR ld/12982: BFD linker LTO plugin makes stack executable by defaul H.J. Lu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).