This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] PR ld/20283: ld: Add a linker configure option --enable-relro

From: Mike Frysinger <vapier at gentoo dot org>
To: Nick Clifton <nickc at redhat dot com>
Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, binutils at sourceware dot org
Date: Wed, 22 Jun 2016 06:43:53 -0400
Subject: Re: [PATCH] PR ld/20283: ld: Add a linker configure option --enable-relro
Authentication-results: sourceware.org; auth=none
References: <20160621221033 dot GA19120 at intel dot com> <52630d65-5f9f-3631-70a5-d566f19070ab at redhat dot com>

On 22 Jun 2016 09:17, Nick Clifton wrote:
> > +# Decide if -z relro should be enabled in ELF linker by default.
> > +ac_default_ld_z_relro=unset
> > +# Provide a configure time option to override our default.
> > +AC_ARG_ENABLE(relro,
> > +	      AS_HELP_STRING([--enable-relro],
> > +	      [enable -z relro in ELF linker by default]),
> > +[case "${enableval}" in
> > +  no)  ac_default_ld_z_relro=0 ;;
> > +esac])dnl
> 
> I think that you ought to support --enable-relro=yes here as well.

i suggested the same thing in the gold version :)
-mike

Attachment: signature.asc
Description: Digital signature

References:
- [PATCH] PR ld/20283: ld: Add a linker configure option --enable-relro
  - From: H.J. Lu
- Re: [PATCH] PR ld/20283: ld: Add a linker configure option --enable-relro
  - From: Nick Clifton

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]