From 7f1be47c1bcd021c5b28a468d443dd49bae705c4 Mon Sep 17 00:00:00 2001 From: James Addison Date: Tue, 9 Mar 2021 13:08:47 +0000 Subject: [PATCH 3/3] src: add .note.GNU-stack section This indicates to GNU binutils that it can unset the executable stack flag on the binary that it is building. PR-URL: https://github.com/nodejs/node/pull/37688 Refs: https://github.com/nodejs/node/issues/17933 Reviewed-By: Daniel Bevenius src: Use %progbits instead of @progbits While @progbits is preferred for most architectures, there are some (notably 32-bit ARM) for which it does not. %progbits is effective everywhere. See https://bugzilla.redhat.com/show_bug.cgi?id=1950528 for more details. Related: https://github.com/nodejs/node/issues/17933 Related: https://github.com/nodejs/node/pull/37688 Signed-off-by: Stephen Gallagher --- src/large_pages/node_text_start.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/large_pages/node_text_start.S b/src/large_pages/node_text_start.S index 3227b62464932ced6d42916a4300ddbd9b4a4fbb..d27dd39cc236f0e6be4e68113bfff7b531a37455 100644 --- a/src/large_pages/node_text_start.S +++ b/src/large_pages/node_text_start.S @@ -1,7 +1,7 @@ #if defined(__ELF__) -.section .note.GNU-stack,"",@progbits +.section .note.GNU-stack,"",%progbits #endif .text .align 0x2000 .global __node_text_start .hidden __node_text_start -- 2.31.1