diff mbox series

[2/4] package/Makefile.in: Add missing options to LDFLAGS for full RELRO build

Message ID 20180425064518.31797-2-stefan.sorensen@spectralink.com
State Superseded
Headers show
Series [1/4] package/Makefile.in: Do not use CPPFLAGS for hardening options | expand

Commit Message

Sørensen, Stefan April 25, 2018, 6:45 a.m. UTC
The options for a full RELRO build should also be added to LDFLAGS.

Signed-off-by: Stefan Sørensen <stefan.sorensen@spectralink.com>
---
 package/Makefile.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Matt Weber May 1, 2018, 3:12 p.m. UTC | #1
Stefan,

On Wed, Apr 25, 2018 at 1:45 AM, Stefan Sørensen
<stefan.sorensen@spectralink.com> wrote:
> The options for a full RELRO build should also be added to LDFLAGS.
>
> Signed-off-by: Stefan Sørensen <stefan.sorensen@spectralink.com>



Tested-by: Matt Weber <matthew.weber@rockwellcollins.com>
diff mbox series

Patch

diff --git a/package/Makefile.in b/package/Makefile.in
index d72a5494ab..1cbf34e0df 100644
--- a/package/Makefile.in
+++ b/package/Makefile.in
@@ -159,7 +159,7 @@  TARGET_HARDENED += $(TARGET_CFLAGS_RELRO)
 TARGET_LDFLAGS += $(TARGET_CFLAGS_RELRO)
 else ifeq ($(BR2_RELRO_FULL),y)
 TARGET_HARDENED += -fPIE $(TARGET_CFLAGS_RELRO_FULL)
-TARGET_LDFLAGS += -pie
+TARGET_LDFLAGS += -pie $(TARGET_CFLAGS_RELRO_FULL)
 endif
 
 ifeq ($(BR2_FORTIFY_SOURCE_1),y)