|
|
56d343 |
commit a3e2ba88eb09c1eed2f7ed6e17660b345464bb90
|
|
|
56d343 |
Author: law <law@138bc75d-0d04-0410-961f-82ee72b054a4>
|
|
|
56d343 |
Date: Wed Sep 20 05:05:12 2017 +0000
|
|
|
56d343 |
|
|
|
56d343 |
2017-09-18 Jeff Law <law@redhat.com>
|
|
|
56d343 |
|
|
|
56d343 |
* explow.c: Include "params.h" and "dumpfile.h".
|
|
|
56d343 |
(anti_adjust_stack_and_probe_stack_clash): New function.
|
|
|
56d343 |
(get_stack_check_protect): Likewise.
|
|
|
56d343 |
(compute_stack_clash_protection_loop_data): Likewise.
|
|
|
56d343 |
(emit_stack_clash_protection_loop_start): Likewise.
|
|
|
56d343 |
(emit_stack_clash_protection_loop_end): Likewise.
|
|
|
56d343 |
(allocate_dynamic_stack_space): Use get_stack_check_protect.
|
|
|
56d343 |
Use anti_adjust_stack_and_probe_stack_clash.
|
|
|
56d343 |
* explow.h (compute_stack_clash_protection_loop_data): Prototype.
|
|
|
56d343 |
(emit_stack_clash_protection_loop_start): Likewise.
|
|
|
56d343 |
(emit_stack_clash_protection_loop_end): Likewise.
|
|
|
56d343 |
* rtl.h (get_stack_check_protect): Prototype.
|
|
|
56d343 |
* target.def (stack_clash_protection_final_dynamic_probe): New hook.
|
|
|
56d343 |
* targhooks.c (default_stack_clash_protection_final_dynamic_probe): New.
|
|
|
56d343 |
* targhooks.h (default_stack_clash_protection_final_dynamic_probe):
|
|
|
56d343 |
Prototype.
|
|
|
56d343 |
* doc/tm.texi.in (TARGET_STACK_CLASH_PROTECTION_FINAL_DYNAMIC_PROBE):
|
|
|
56d343 |
Add @hook.
|
|
|
56d343 |
* doc/tm.texi: Rebuilt.
|
|
|
56d343 |
* config/alpha/alpha.c (alpha_expand_prologue): Likewise.
|
|
|
56d343 |
* config/i386/i386.c (ix86_expand_prologue): Likewise.
|
|
|
56d343 |
* config/ia64/ia64.c (ia64_expand_prologue): Likewise.
|
|
|
56d343 |
* config/mips/mips.c (mips_expand_prologue): Likewise.
|
|
|
56d343 |
* config/rs6000/rs6000.c (rs6000_emit_prologue): Likewise.
|
|
|
56d343 |
* config/sparc/sparc.c (sparc_expand_prologue): Likewise.
|
|
|
56d343 |
(sparc_flat_expand_prologue): Likewise.
|
|
|
56d343 |
|
|
|
56d343 |
* gcc.dg/stack-check-3.c: New test.
|
|
|
56d343 |
|
|
|
56d343 |
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@252995 138bc75d-0d04-0410-961f-82ee72b054a4
|
|
|
56d343 |
|
|
|
56d343 |
diff --git a/gcc/config/alpha/alpha.c b/gcc/config/alpha/alpha.c
|
|
|
56d343 |
index 2874b8454a9..5402f5213d6 100644
|
|
|
56d343 |
--- a/gcc/config/alpha/alpha.c
|
|
|
56d343 |
+++ b/gcc/config/alpha/alpha.c
|
|
|
56d343 |
@@ -7625,7 +7625,7 @@ alpha_expand_prologue (void)
|
|
|
56d343 |
|
|
|
56d343 |
probed_size = frame_size;
|
|
|
56d343 |
if (flag_stack_check)
|
|
|
56d343 |
- probed_size += STACK_CHECK_PROTECT;
|
|
|
56d343 |
+ probed_size += get_stack_check_protect ();
|
|
|
56d343 |
|
|
|
56d343 |
if (probed_size <= 32768)
|
|
|
56d343 |
{
|
|
|
56d343 |
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
|
|
|
56d343 |
index e36726ba722..d996fd160e8 100644
|
|
|
56d343 |
--- a/gcc/config/i386/i386.c
|
|
|
56d343 |
+++ b/gcc/config/i386/i386.c
|
|
|
56d343 |
@@ -10544,12 +10544,12 @@ ix86_expand_prologue (void)
|
|
|
56d343 |
HOST_WIDE_INT size = allocate;
|
|
|
56d343 |
|
|
|
56d343 |
if (TARGET_64BIT && size >= (HOST_WIDE_INT) 0x80000000)
|
|
|
56d343 |
- size = 0x80000000 - STACK_CHECK_PROTECT - 1;
|
|
|
56d343 |
+ size = 0x80000000 - get_stack_check_protect () - 1;
|
|
|
56d343 |
|
|
|
56d343 |
if (TARGET_STACK_PROBE)
|
|
|
56d343 |
- ix86_emit_probe_stack_range (0, size + STACK_CHECK_PROTECT);
|
|
|
56d343 |
+ ix86_emit_probe_stack_range (0, size + get_stack_check_protect ());
|
|
|
56d343 |
else
|
|
|
56d343 |
- ix86_emit_probe_stack_range (STACK_CHECK_PROTECT, size);
|
|
|
56d343 |
+ ix86_emit_probe_stack_range (get_stack_check_protect (), size);
|
|
|
56d343 |
}
|
|
|
56d343 |
}
|
|
|
56d343 |
|
|
|
56d343 |
diff --git a/gcc/config/ia64/ia64.c b/gcc/config/ia64/ia64.c
|
|
|
56d343 |
index 50bbad6661c..390983936e8 100644
|
|
|
56d343 |
--- a/gcc/config/ia64/ia64.c
|
|
|
56d343 |
+++ b/gcc/config/ia64/ia64.c
|
|
|
56d343 |
@@ -3435,7 +3435,7 @@ ia64_expand_prologue (void)
|
|
|
56d343 |
current_function_static_stack_size = current_frame_info.total_size;
|
|
|
56d343 |
|
|
|
56d343 |
if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK)
|
|
|
56d343 |
- ia64_emit_probe_stack_range (STACK_CHECK_PROTECT,
|
|
|
56d343 |
+ ia64_emit_probe_stack_range (get_stack_check_protect (),
|
|
|
56d343 |
current_frame_info.total_size,
|
|
|
56d343 |
current_frame_info.n_input_regs
|
|
|
56d343 |
+ current_frame_info.n_local_regs);
|
|
|
56d343 |
diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c
|
|
|
56d343 |
index 41c5d6b6b1f..9b7eb678f19 100644
|
|
|
56d343 |
--- a/gcc/config/mips/mips.c
|
|
|
56d343 |
+++ b/gcc/config/mips/mips.c
|
|
|
56d343 |
@@ -10746,7 +10746,7 @@ mips_expand_prologue (void)
|
|
|
56d343 |
current_function_static_stack_size = size;
|
|
|
56d343 |
|
|
|
56d343 |
if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK && size)
|
|
|
56d343 |
- mips_emit_probe_stack_range (STACK_CHECK_PROTECT, size);
|
|
|
56d343 |
+ mips_emit_probe_stack_range (get_stack_check_protect (), size);
|
|
|
56d343 |
|
|
|
56d343 |
/* Save the registers. Allocate up to MIPS_MAX_FIRST_STACK_STEP
|
|
|
56d343 |
bytes beforehand; this is enough to cover the register save area
|
|
|
56d343 |
diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
|
|
|
56d343 |
index 15583055895..a9052c6becf 100644
|
|
|
56d343 |
--- a/gcc/config/rs6000/rs6000.c
|
|
|
56d343 |
+++ b/gcc/config/rs6000/rs6000.c
|
|
|
56d343 |
@@ -23214,7 +23214,8 @@ rs6000_emit_prologue (void)
|
|
|
56d343 |
current_function_static_stack_size = info->total_size;
|
|
|
56d343 |
|
|
|
56d343 |
if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK && info->total_size)
|
|
|
56d343 |
- rs6000_emit_probe_stack_range (STACK_CHECK_PROTECT, info->total_size);
|
|
|
56d343 |
+ rs6000_emit_probe_stack_range (get_stack_check_protect (),
|
|
|
56d343 |
+ info->total_size);
|
|
|
56d343 |
|
|
|
56d343 |
if (TARGET_FIX_AND_CONTINUE)
|
|
|
56d343 |
{
|
|
|
56d343 |
diff --git a/gcc/config/sparc/sparc.c b/gcc/config/sparc/sparc.c
|
|
|
56d343 |
index e5d326cdf23..e5e93c80261 100644
|
|
|
56d343 |
--- a/gcc/config/sparc/sparc.c
|
|
|
56d343 |
+++ b/gcc/config/sparc/sparc.c
|
|
|
56d343 |
@@ -5431,7 +5431,7 @@ sparc_expand_prologue (void)
|
|
|
56d343 |
current_function_static_stack_size = size;
|
|
|
56d343 |
|
|
|
56d343 |
if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK && size)
|
|
|
56d343 |
- sparc_emit_probe_stack_range (STACK_CHECK_PROTECT, size);
|
|
|
56d343 |
+ sparc_emit_probe_stack_range (get_stack_check_protect (), size);
|
|
|
56d343 |
|
|
|
56d343 |
if (size == 0)
|
|
|
56d343 |
; /* do nothing. */
|
|
|
56d343 |
@@ -5533,7 +5533,7 @@ sparc_flat_expand_prologue (void)
|
|
|
56d343 |
current_function_static_stack_size = size;
|
|
|
56d343 |
|
|
|
56d343 |
if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK && size)
|
|
|
56d343 |
- sparc_emit_probe_stack_range (STACK_CHECK_PROTECT, size);
|
|
|
56d343 |
+ sparc_emit_probe_stack_range (get_stack_check_protect (), size);
|
|
|
56d343 |
|
|
|
56d343 |
if (sparc_save_local_in_regs_p)
|
|
|
56d343 |
emit_save_or_restore_local_in_regs (stack_pointer_rtx, SPARC_STACK_BIAS,
|
|
|
56d343 |
diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
|
|
|
56d343 |
index 6b18a2724bc..eeef757bf5b 100644
|
|
|
56d343 |
--- a/gcc/doc/tm.texi
|
|
|
56d343 |
+++ b/gcc/doc/tm.texi
|
|
|
56d343 |
@@ -3571,6 +3571,10 @@ GCC computed the default from the values of the above macros and you will
|
|
|
56d343 |
normally not need to override that default.
|
|
|
56d343 |
@end defmac
|
|
|
56d343 |
|
|
|
56d343 |
+@deftypefn {Target Hook} bool TARGET_STACK_CLASH_PROTECTION_FINAL_DYNAMIC_PROBE (rtx @var{residual})
|
|
|
56d343 |
+Some targets make optimistic assumptions about the state of stack probing when they emit their prologues. On such targets a probe into the end of any dynamically allocated space is likely required for safety against stack clash style attacks. Define this variable to return nonzero if such a probe is required or zero otherwise. You need not define this macro if it would always have the value zero.
|
|
|
56d343 |
+@end deftypefn
|
|
|
56d343 |
+
|
|
|
56d343 |
@need 2000
|
|
|
56d343 |
@node Frame Registers
|
|
|
56d343 |
@subsection Registers That Address the Stack Frame
|
|
|
56d343 |
diff --git a/gcc/doc/tm.texi.in b/gcc/doc/tm.texi.in
|
|
|
56d343 |
index 7d0b3c73b2f..6707ca87236 100644
|
|
|
56d343 |
--- a/gcc/doc/tm.texi.in
|
|
|
56d343 |
+++ b/gcc/doc/tm.texi.in
|
|
|
56d343 |
@@ -3539,6 +3539,8 @@ GCC computed the default from the values of the above macros and you will
|
|
|
56d343 |
normally not need to override that default.
|
|
|
56d343 |
@end defmac
|
|
|
56d343 |
|
|
|
56d343 |
+@hook TARGET_STACK_CLASH_PROTECTION_FINAL_DYNAMIC_PROBE
|
|
|
56d343 |
+
|
|
|
56d343 |
@need 2000
|
|
|
56d343 |
@node Frame Registers
|
|
|
56d343 |
@subsection Registers That Address the Stack Frame
|
|
|
56d343 |
diff --git a/gcc/explow.c b/gcc/explow.c
|
|
|
56d343 |
index 7da8bc75f19..2526e8513b7 100644
|
|
|
56d343 |
--- a/gcc/explow.c
|
|
|
56d343 |
+++ b/gcc/explow.c
|
|
|
56d343 |
@@ -40,8 +40,11 @@ along with GCC; see the file COPYING3. If not see
|
|
|
56d343 |
#include "target.h"
|
|
|
56d343 |
#include "common/common-target.h"
|
|
|
56d343 |
#include "output.h"
|
|
|
56d343 |
+#include "params.h"
|
|
|
56d343 |
+#include "dumpfile.h"
|
|
|
56d343 |
|
|
|
56d343 |
static rtx break_out_memory_refs (rtx);
|
|
|
56d343 |
+static void anti_adjust_stack_and_probe_stack_clash (rtx);
|
|
|
56d343 |
|
|
|
56d343 |
|
|
|
56d343 |
/* Truncate and perhaps sign-extend C as appropriate for MODE. */
|
|
|
56d343 |
@@ -1140,6 +1143,29 @@ update_nonlocal_goto_save_area (void)
|
|
|
56d343 |
emit_stack_save (SAVE_NONLOCAL, &r_save);
|
|
|
56d343 |
}
|
|
|
56d343 |
|
|
|
56d343 |
+/* Return the number of bytes to "protect" on the stack for -fstack-check.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ "protect" in the context of -fstack-check means how many bytes we
|
|
|
56d343 |
+ should always ensure are available on the stack. More importantly
|
|
|
56d343 |
+ this is how many bytes are skipped when probing the stack.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ On some targets we want to reuse the -fstack-check prologue support
|
|
|
56d343 |
+ to give a degree of protection against stack clashing style attacks.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ In that scenario we do not want to skip bytes before probing as that
|
|
|
56d343 |
+ would render the stack clash protections useless.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ So we never use STACK_CHECK_PROTECT directly. Instead we indirect though
|
|
|
56d343 |
+ this helper which allows us to provide different values for
|
|
|
56d343 |
+ -fstack-check and -fstack-clash-protection. */
|
|
|
56d343 |
+HOST_WIDE_INT
|
|
|
56d343 |
+get_stack_check_protect (void)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ if (flag_stack_clash_protection)
|
|
|
56d343 |
+ return 0;
|
|
|
56d343 |
+ return STACK_CHECK_PROTECT;
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
/* Return an rtx representing the address of an area of memory dynamically
|
|
|
56d343 |
pushed on the stack.
|
|
|
56d343 |
|
|
|
56d343 |
@@ -1393,7 +1419,7 @@ allocate_dynamic_stack_space (rtx size, unsigned size_align,
|
|
|
56d343 |
probe_stack_range (STACK_OLD_CHECK_PROTECT + STACK_CHECK_MAX_FRAME_SIZE,
|
|
|
56d343 |
size);
|
|
|
56d343 |
else if (flag_stack_check == STATIC_BUILTIN_STACK_CHECK)
|
|
|
56d343 |
- probe_stack_range (STACK_CHECK_PROTECT, size);
|
|
|
56d343 |
+ probe_stack_range (get_stack_check_protect (), size);
|
|
|
56d343 |
|
|
|
56d343 |
/* Don't let anti_adjust_stack emit notes. */
|
|
|
56d343 |
suppress_reg_args_size = true;
|
|
|
56d343 |
@@ -1451,6 +1477,8 @@ allocate_dynamic_stack_space (rtx size, unsigned size_align,
|
|
|
56d343 |
|
|
|
56d343 |
if (flag_stack_check && STACK_CHECK_MOVING_SP)
|
|
|
56d343 |
anti_adjust_stack_and_probe (size, false);
|
|
|
56d343 |
+ else if (flag_stack_clash_protection)
|
|
|
56d343 |
+ anti_adjust_stack_and_probe_stack_clash (size);
|
|
|
56d343 |
else
|
|
|
56d343 |
anti_adjust_stack (size);
|
|
|
56d343 |
|
|
|
56d343 |
@@ -1712,6 +1740,219 @@ probe_stack_range (HOST_WIDE_INT first, rtx size)
|
|
|
56d343 |
}
|
|
|
56d343 |
}
|
|
|
56d343 |
|
|
|
56d343 |
+/* Compute parameters for stack clash probing a dynamic stack
|
|
|
56d343 |
+ allocation of SIZE bytes.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ We compute ROUNDED_SIZE, LAST_ADDR, RESIDUAL and PROBE_INTERVAL.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ Additionally we conditionally dump the type of probing that will
|
|
|
56d343 |
+ be needed given the values computed. */
|
|
|
56d343 |
+
|
|
|
56d343 |
+void
|
|
|
56d343 |
+compute_stack_clash_protection_loop_data (rtx *rounded_size, rtx *last_addr,
|
|
|
56d343 |
+ rtx *residual,
|
|
|
56d343 |
+ HOST_WIDE_INT *probe_interval,
|
|
|
56d343 |
+ rtx size)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ /* Round SIZE down to STACK_CLASH_PROTECTION_PROBE_INTERVAL */
|
|
|
56d343 |
+ *probe_interval
|
|
|
56d343 |
+ = 1 << PARAM_VALUE (PARAM_STACK_CLASH_PROTECTION_PROBE_INTERVAL);
|
|
|
56d343 |
+ *rounded_size = simplify_gen_binary (AND, Pmode, size,
|
|
|
56d343 |
+ GEN_INT (-*probe_interval));
|
|
|
56d343 |
+
|
|
|
56d343 |
+ /* Compute the value of the stack pointer for the last iteration.
|
|
|
56d343 |
+ It's just SP + ROUNDED_SIZE. */
|
|
|
56d343 |
+ rtx rounded_size_op = force_operand (*rounded_size, NULL_RTX);
|
|
|
56d343 |
+ *last_addr = force_operand (gen_rtx_fmt_ee (STACK_GROW_OP, Pmode,
|
|
|
56d343 |
+ stack_pointer_rtx,
|
|
|
56d343 |
+ rounded_size_op),
|
|
|
56d343 |
+ NULL_RTX);
|
|
|
56d343 |
+
|
|
|
56d343 |
+ /* Compute any residuals not allocated by the loop above. Residuals
|
|
|
56d343 |
+ are just the ROUNDED_SIZE - SIZE. */
|
|
|
56d343 |
+ *residual = simplify_gen_binary (MINUS, Pmode, size, *rounded_size);
|
|
|
56d343 |
+
|
|
|
56d343 |
+ /* Dump key information to make writing tests easy. */
|
|
|
56d343 |
+ if (dump_file)
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ if (*rounded_size == CONST0_RTX (Pmode))
|
|
|
56d343 |
+ fprintf (dump_file,
|
|
|
56d343 |
+ "Stack clash skipped dynamic allocation and probing loop.\n");
|
|
|
56d343 |
+ else if (GET_CODE (*rounded_size) == CONST_INT
|
|
|
56d343 |
+ && INTVAL (*rounded_size) <= 4 * *probe_interval)
|
|
|
56d343 |
+ fprintf (dump_file,
|
|
|
56d343 |
+ "Stack clash dynamic allocation and probing inline.\n");
|
|
|
56d343 |
+ else if (GET_CODE (*rounded_size) == CONST_INT)
|
|
|
56d343 |
+ fprintf (dump_file,
|
|
|
56d343 |
+ "Stack clash dynamic allocation and probing in "
|
|
|
56d343 |
+ "rotated loop.\n");
|
|
|
56d343 |
+ else
|
|
|
56d343 |
+ fprintf (dump_file,
|
|
|
56d343 |
+ "Stack clash dynamic allocation and probing in loop.\n");
|
|
|
56d343 |
+
|
|
|
56d343 |
+ if (*residual != CONST0_RTX (Pmode))
|
|
|
56d343 |
+ fprintf (dump_file,
|
|
|
56d343 |
+ "Stack clash dynamic allocation and probing residuals.\n");
|
|
|
56d343 |
+ else
|
|
|
56d343 |
+ fprintf (dump_file,
|
|
|
56d343 |
+ "Stack clash skipped dynamic allocation and "
|
|
|
56d343 |
+ "probing residuals.\n");
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Emit the start of an allocate/probe loop for stack
|
|
|
56d343 |
+ clash protection.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ LOOP_LAB and END_LAB are returned for use when we emit the
|
|
|
56d343 |
+ end of the loop.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ LAST addr is the value for SP which stops the loop. */
|
|
|
56d343 |
+void
|
|
|
56d343 |
+emit_stack_clash_protection_probe_loop_start (rtx *loop_lab,
|
|
|
56d343 |
+ rtx *end_lab,
|
|
|
56d343 |
+ rtx last_addr,
|
|
|
56d343 |
+ bool rotated)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ /* Essentially we want to emit any setup code, the top of loop
|
|
|
56d343 |
+ label and the comparison at the top of the loop. */
|
|
|
56d343 |
+ *loop_lab = gen_label_rtx ();
|
|
|
56d343 |
+ *end_lab = gen_label_rtx ();
|
|
|
56d343 |
+
|
|
|
56d343 |
+ emit_label (*loop_lab);
|
|
|
56d343 |
+ if (!rotated)
|
|
|
56d343 |
+ emit_cmp_and_jump_insns (stack_pointer_rtx, last_addr, EQ, NULL_RTX,
|
|
|
56d343 |
+ Pmode, 1, *end_lab);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Emit the end of a stack clash probing loop.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ This consists of just the jump back to LOOP_LAB and
|
|
|
56d343 |
+ emitting END_LOOP after the loop. */
|
|
|
56d343 |
+
|
|
|
56d343 |
+void
|
|
|
56d343 |
+emit_stack_clash_protection_probe_loop_end (rtx loop_lab, rtx end_loop,
|
|
|
56d343 |
+ rtx last_addr, bool rotated)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ if (rotated)
|
|
|
56d343 |
+ emit_cmp_and_jump_insns (stack_pointer_rtx, last_addr, NE, NULL_RTX,
|
|
|
56d343 |
+ Pmode, 1, loop_lab);
|
|
|
56d343 |
+ else
|
|
|
56d343 |
+ emit_jump (loop_lab);
|
|
|
56d343 |
+
|
|
|
56d343 |
+ emit_label (end_loop);
|
|
|
56d343 |
+
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Adjust the stack pointer by minus SIZE (an rtx for a number of bytes)
|
|
|
56d343 |
+ while probing it. This pushes when SIZE is positive. SIZE need not
|
|
|
56d343 |
+ be constant.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ This is subtly different than anti_adjust_stack_and_probe to try and
|
|
|
56d343 |
+ prevent stack-clash attacks
|
|
|
56d343 |
+
|
|
|
56d343 |
+ 1. It must assume no knowledge of the probing state, any allocation
|
|
|
56d343 |
+ must probe.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ Consider the case of a 1 byte alloca in a loop. If the sum of the
|
|
|
56d343 |
+ allocations is large, then this could be used to jump the guard if
|
|
|
56d343 |
+ probes were not emitted.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ 2. It never skips probes, whereas anti_adjust_stack_and_probe will
|
|
|
56d343 |
+ skip probes on the first couple PROBE_INTERVALs on the assumption
|
|
|
56d343 |
+ they're done elsewhere.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ 3. It only allocates and probes SIZE bytes, it does not need to
|
|
|
56d343 |
+ allocate/probe beyond that because this probing style does not
|
|
|
56d343 |
+ guarantee signal handling capability if the guard is hit. */
|
|
|
56d343 |
+
|
|
|
56d343 |
+static void
|
|
|
56d343 |
+anti_adjust_stack_and_probe_stack_clash (rtx size)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ /* First ensure SIZE is Pmode. */
|
|
|
56d343 |
+ if (GET_MODE (size) != VOIDmode && GET_MODE (size) != Pmode)
|
|
|
56d343 |
+ size = convert_to_mode (Pmode, size, 1);
|
|
|
56d343 |
+
|
|
|
56d343 |
+ /* We can get here with a constant size on some targets. */
|
|
|
56d343 |
+ rtx rounded_size, last_addr, residual;
|
|
|
56d343 |
+ HOST_WIDE_INT probe_interval;
|
|
|
56d343 |
+ compute_stack_clash_protection_loop_data (&rounded_size, &last_addr,
|
|
|
56d343 |
+ &residual, &probe_interval, size);
|
|
|
56d343 |
+
|
|
|
56d343 |
+ if (rounded_size != CONST0_RTX (Pmode))
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ if (INTVAL (rounded_size) <= 4 * probe_interval)
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ for (HOST_WIDE_INT i = 0;
|
|
|
56d343 |
+ i < INTVAL (rounded_size);
|
|
|
56d343 |
+ i += probe_interval)
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ anti_adjust_stack (GEN_INT (probe_interval));
|
|
|
56d343 |
+
|
|
|
56d343 |
+ /* The prologue does not probe residuals. Thus the offset
|
|
|
56d343 |
+ here to probe just beyond what the prologue had already
|
|
|
56d343 |
+ allocated. */
|
|
|
56d343 |
+ emit_stack_probe (plus_constant (Pmode, stack_pointer_rtx,
|
|
|
56d343 |
+ (probe_interval
|
|
|
56d343 |
+ - GET_MODE_SIZE (word_mode))));
|
|
|
56d343 |
+ emit_insn (gen_blockage ());
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+ else
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ rtx loop_lab, end_loop;
|
|
|
56d343 |
+ bool rotate_loop = GET_CODE (rounded_size) == CONST_INT;
|
|
|
56d343 |
+ emit_stack_clash_protection_probe_loop_start (&loop_lab, &end_loop,
|
|
|
56d343 |
+ last_addr, rotate_loop);
|
|
|
56d343 |
+
|
|
|
56d343 |
+ anti_adjust_stack (GEN_INT (probe_interval));
|
|
|
56d343 |
+
|
|
|
56d343 |
+ /* The prologue does not probe residuals. Thus the offset here
|
|
|
56d343 |
+ to probe just beyond what the prologue had already allocated. */
|
|
|
56d343 |
+ emit_stack_probe (plus_constant (Pmode, stack_pointer_rtx,
|
|
|
56d343 |
+ (probe_interval
|
|
|
56d343 |
+ - GET_MODE_SIZE (word_mode))));
|
|
|
56d343 |
+
|
|
|
56d343 |
+ emit_stack_clash_protection_probe_loop_end (loop_lab, end_loop,
|
|
|
56d343 |
+ last_addr, rotate_loop);
|
|
|
56d343 |
+ emit_insn (gen_blockage ());
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+
|
|
|
56d343 |
+ if (residual != CONST0_RTX (Pmode))
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ rtx x = force_reg (Pmode, plus_constant (Pmode, residual,
|
|
|
56d343 |
+ -GET_MODE_SIZE (word_mode)));
|
|
|
56d343 |
+ anti_adjust_stack (residual);
|
|
|
56d343 |
+ emit_stack_probe (gen_rtx_PLUS (Pmode, stack_pointer_rtx, x));
|
|
|
56d343 |
+ emit_insn (gen_blockage ());
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+
|
|
|
56d343 |
+ /* Some targets make optimistic assumptions in their prologues about
|
|
|
56d343 |
+ how the caller may have probed the stack. Make sure we honor
|
|
|
56d343 |
+ those assumptions when needed. */
|
|
|
56d343 |
+ if (size != CONST0_RTX (Pmode)
|
|
|
56d343 |
+ && targetm.stack_clash_protection_final_dynamic_probe (residual))
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ /* Ideally we would just probe at *sp. However, if SIZE is not
|
|
|
56d343 |
+ a compile-time constant, but is zero at runtime, then *sp
|
|
|
56d343 |
+ might hold live data. So probe at *sp if we know that
|
|
|
56d343 |
+ an allocation was made, otherwise probe into the red zone
|
|
|
56d343 |
+ which is obviously undesirable. */
|
|
|
56d343 |
+ if (GET_CODE (size) == CONST_INT)
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ emit_stack_probe (stack_pointer_rtx);
|
|
|
56d343 |
+ emit_insn (gen_blockage ());
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+ else
|
|
|
56d343 |
+ {
|
|
|
56d343 |
+ emit_stack_probe (plus_constant (Pmode, stack_pointer_rtx,
|
|
|
56d343 |
+ -GET_MODE_SIZE (word_mode)));
|
|
|
56d343 |
+ emit_insn (gen_blockage ());
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+ }
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+
|
|
|
56d343 |
/* Adjust the stack pointer by minus SIZE (an rtx for a number of bytes)
|
|
|
56d343 |
while probing it. This pushes when SIZE is positive. SIZE need not
|
|
|
56d343 |
be constant. If ADJUST_BACK is true, adjust back the stack pointer
|
|
|
56d343 |
diff --git a/gcc/rtl.h b/gcc/rtl.h
|
|
|
56d343 |
index 91f3387c701..ab8ec27418d 100644
|
|
|
56d343 |
--- a/gcc/rtl.h
|
|
|
56d343 |
+++ b/gcc/rtl.h
|
|
|
56d343 |
@@ -1756,6 +1756,17 @@ extern int currently_expanding_to_rtl;
|
|
|
56d343 |
/* In explow.c */
|
|
|
56d343 |
extern HOST_WIDE_INT trunc_int_for_mode (HOST_WIDE_INT, enum machine_mode);
|
|
|
56d343 |
extern rtx plus_constant (enum machine_mode, rtx, HOST_WIDE_INT);
|
|
|
56d343 |
+extern HOST_WIDE_INT get_stack_check_protect (void);
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Support for building allocation/probing loops for stack-clash
|
|
|
56d343 |
+ protection of dyamically allocated stack space. */
|
|
|
56d343 |
+extern void compute_stack_clash_protection_loop_data (rtx *, rtx *, rtx *,
|
|
|
56d343 |
+ HOST_WIDE_INT *, rtx);
|
|
|
56d343 |
+extern void emit_stack_clash_protection_probe_loop_start (rtx *, rtx *,
|
|
|
56d343 |
+ rtx, bool);
|
|
|
56d343 |
+extern void emit_stack_clash_protection_probe_loop_end (rtx, rtx,
|
|
|
56d343 |
+ rtx, bool);
|
|
|
56d343 |
+
|
|
|
56d343 |
|
|
|
56d343 |
/* In rtl.c */
|
|
|
56d343 |
extern rtx rtx_alloc_stat (RTX_CODE MEM_STAT_DECL);
|
|
|
56d343 |
diff --git a/gcc/target.def b/gcc/target.def
|
|
|
56d343 |
index 4d6081c3121..eb2bd46f7a1 100644
|
|
|
56d343 |
--- a/gcc/target.def
|
|
|
56d343 |
+++ b/gcc/target.def
|
|
|
56d343 |
@@ -2580,6 +2580,13 @@ DEFHOOK
|
|
|
56d343 |
void, (void),
|
|
|
56d343 |
hook_void_void)
|
|
|
56d343 |
|
|
|
56d343 |
+DEFHOOK
|
|
|
56d343 |
+(stack_clash_protection_final_dynamic_probe,
|
|
|
56d343 |
+ "Some targets make optimistic assumptions about the state of stack probing when they emit their prologues. On such targets a probe into the end of any dynamically allocated space is likely required for safety against stack clash style attacks. Define this variable to return nonzero if such a probe is required or zero otherwise. You need not define this macro if it would always have the value zero.",
|
|
|
56d343 |
+ bool, (rtx residual),
|
|
|
56d343 |
+ default_stack_clash_protection_final_dynamic_probe)
|
|
|
56d343 |
+
|
|
|
56d343 |
+
|
|
|
56d343 |
/* Functions specific to the C family of frontends. */
|
|
|
56d343 |
#undef HOOK_PREFIX
|
|
|
56d343 |
#define HOOK_PREFIX "TARGET_C_"
|
|
|
56d343 |
diff --git a/gcc/targhooks.c b/gcc/targhooks.c
|
|
|
56d343 |
index f6aa9907225..be23875538d 100644
|
|
|
56d343 |
--- a/gcc/targhooks.c
|
|
|
56d343 |
+++ b/gcc/targhooks.c
|
|
|
56d343 |
@@ -1557,4 +1557,10 @@ default_canonicalize_comparison (int *, rtx *, rtx *, bool)
|
|
|
56d343 |
{
|
|
|
56d343 |
}
|
|
|
56d343 |
|
|
|
56d343 |
+bool
|
|
|
56d343 |
+default_stack_clash_protection_final_dynamic_probe (rtx residual ATTRIBUTE_UNUSED)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ return 0;
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
#include "gt-targhooks.h"
|
|
|
56d343 |
diff --git a/gcc/targhooks.h b/gcc/targhooks.h
|
|
|
56d343 |
index b64274d3ff9..4acf33fae08 100644
|
|
|
56d343 |
--- a/gcc/targhooks.h
|
|
|
56d343 |
+++ b/gcc/targhooks.h
|
|
|
56d343 |
@@ -195,3 +195,4 @@ extern const char *default_pch_valid_p (const void *, size_t);
|
|
|
56d343 |
extern void default_asm_output_ident_directive (const char*);
|
|
|
56d343 |
|
|
|
56d343 |
extern bool default_member_type_forces_blk (const_tree, enum machine_mode);
|
|
|
56d343 |
+extern bool default_stack_clash_protection_final_dynamic_probe (rtx);
|
|
|
56d343 |
diff --git a/gcc/testsuite/gcc.dg/stack-check-3.c b/gcc/testsuite/gcc.dg/stack-check-3.c
|
|
|
56d343 |
new file mode 100644
|
|
|
56d343 |
index 00000000000..58fb65649ee
|
|
|
56d343 |
--- /dev/null
|
|
|
56d343 |
+++ b/gcc/testsuite/gcc.dg/stack-check-3.c
|
|
|
56d343 |
@@ -0,0 +1,86 @@
|
|
|
56d343 |
+/* The goal here is to ensure that dynamic allocations via vlas or
|
|
|
56d343 |
+ alloca calls receive probing.
|
|
|
56d343 |
+
|
|
|
56d343 |
+ Scanning the RTL or assembly code seems like insanity here as does
|
|
|
56d343 |
+ checking for particular allocation sizes and probe offsets. For
|
|
|
56d343 |
+ now we just verify that there's an allocation + probe loop and
|
|
|
56d343 |
+ residual allocation + probe for f?. */
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* { dg-do compile } */
|
|
|
56d343 |
+/* { dg-options "-O2 -fstack-clash-protection -fdump-rtl-expand -fno-optimize-sibling-calls --param stack-clash-protection-probe-interval=4096 --param stack-clash-protection-guard-size=4096" } */
|
|
|
56d343 |
+/* { dg-require-effective-target supports_stack_clash_protection } */
|
|
|
56d343 |
+
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+foo (char *p)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ asm volatile ("" : : "r" (p) : "memory");
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Simple VLA, no other locals. */
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+f0 (int x)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ char vla[x];
|
|
|
56d343 |
+ foo (vla);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Simple VLA, small local frame. */
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+f1 (int x)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ char locals[128];
|
|
|
56d343 |
+ char vla[x];
|
|
|
56d343 |
+ foo (vla);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Small constant alloca, no other locals. */
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+f2 (int x)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ char *vla = __builtin_alloca (128);
|
|
|
56d343 |
+ foo (vla);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Big constant alloca, small local frame. */
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+f3 (int x)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ char locals[128];
|
|
|
56d343 |
+ char *vla = __builtin_alloca (16384);
|
|
|
56d343 |
+ foo (vla);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Big constant alloca, small local frame. */
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+f3a (int x)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ char locals[128];
|
|
|
56d343 |
+ char *vla = __builtin_alloca (32768);
|
|
|
56d343 |
+ foo (vla);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Nonconstant alloca, no other locals. */
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+f4 (int x)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ char *vla = __builtin_alloca (x);
|
|
|
56d343 |
+ foo (vla);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* Nonconstant alloca, small local frame. */
|
|
|
56d343 |
+__attribute__((noinline, noclone)) void
|
|
|
56d343 |
+f5 (int x)
|
|
|
56d343 |
+{
|
|
|
56d343 |
+ char locals[128];
|
|
|
56d343 |
+ char *vla = __builtin_alloca (x);
|
|
|
56d343 |
+ foo (vla);
|
|
|
56d343 |
+}
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* { dg-final { scan-rtl-dump-times "allocation and probing residuals" 7 "expand" } } */
|
|
|
56d343 |
+
|
|
|
56d343 |
+
|
|
|
56d343 |
+/* { dg-final { scan-rtl-dump-times "allocation and probing in loop" 7 "expand" { target callee_realigns_stack } } } */
|
|
|
56d343 |
+/* { dg-final { scan-rtl-dump-times "allocation and probing in loop" 4 "expand" { target { ! callee_realigns_stack } } } } */
|
|
|
56d343 |
+/* { dg-final { scan-rtl-dump-times "allocation and probing in rotated loop" 1 "expand" { target { ! callee_realigns_stack } } } } */
|
|
|
56d343 |
+/* { dg-final { scan-rtl-dump-times "allocation and probing inline" 1 "expand" { target { ! callee_realigns_stack } } } } */
|
|
|
56d343 |
+/* { dg-final { scan-rtl-dump-times "skipped dynamic allocation and probing loop" 1 "expand" { target { ! callee_realigns_stack } } } } */
|